-
Notifications
You must be signed in to change notification settings - Fork 178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Design review #1923
Comments
Updated with the notes by @nimishavijay and me from the additional 2 design reviews we had beginning of 2021. :) |
So @jancborchardt and I went through this list and thought that having a view/edit mode would be a good place to start, along with some more minor visual changes mainly in the content. View mode:
Edit mode:
This could be a good place to start, moving forward we could make changes to more sections. |
By the way - also see #2699 where I created a set of custom CSS rules which can be used with the CustomCSS app to modify the layout of the contacts app. I am also willing to participate and provide an updated layout for the app itself and not only as workaround using CSS, but before doing so it would be nice to learn what plans you may have already. |
There are no plans for the items in this ticket. They can be picked up at any time. Ideally they are tackled in individual fixes to keep the changes trackable and reviewable. |
I am also willing to help. |
The first step would be to get a working development environment up and running, so one can work on this. Also see https://nextcloud.com/developer/ and the links to the tutorials like "Setting up a development environment". The contacts app can also just be checked out out from Github and built locally. Experience in Vue.js is also helpful. |
I will start from the more complicated ones by opening a separate issue for them, if the points above dont have an open issue, it means im not working on it. So, if someone wants to pick up the easy ones, feel free to do so |
@GretaD, if you're able to edit this issue, you can easily create new ones simply by clicking on them. |
Can you link the issues here, so it is easy to find them? Thank you! |
Thanks for the tip. Unfortunately those tickets are not very useful. Github just takes the single of the checklist here and creates an empty ticket with a subject. Therefore we prefer to create proper tickets manually and replace the checklist items with a ticket link.
We will link the tickets as per above. @GretaD maybe you can tackle this preparation work first. Then we have a set of small tickets and distribute them across a bunch of people. Most work should be independent and can be worked on in parallel. |
Worked on updated mockups with @jancborchardt and came up with this: Includes 2 bigger enhancements which can be worked on separately as well:
Enhancements related to view/edit mode, including points from the original design review and previous mockups:
View mode:
Edit mode:
Nice to have/future enhancements
cc @jancborchardt and @GretaD :) Moved into #3284 |
Hello and thank you for offering to help :), |
Because all major changes for Nextcloud 27 are done. The rest of the design improvements are done during stabilization phase when time allows. |
Closing as done, lots have changed since the review and it’s best to do a separate follow-up review. :) |
hi, I suggest to unfold this roll down menu to a fixed list of all field types thanks for your work and concern |
Overall
Sort favorite contacts first in list #105
Papercuts and bugs
Navigation
➡️ To be managed on the vue components globally
Contact list
Edit-View Mode Slider #311
Search
Detail view
Layout improvements
Responsive columns --> fixed to maximum 2 instead of 6(Will be fixed to a single column)Settings
Import
Show more details about the failing reason of an import #990
Circles integration
name
"Opens a modal directly for putting in the name
Has a very short explanation on what "Circles" are, below the Circle name input
Profiles integration
Acceptance criteria
Work packages for Nextcloud 27
The text was updated successfully, but these errors were encountered: