-
Notifications
You must be signed in to change notification settings - Fork 135
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
Put star on the left side of the note #2
Comments
(Then it’s also not that close to the delete button anymore which is a bit safer. ;) |
@jancborchardt Hmmm. Typically, the icon left to the text in the navigation sidebar is a visual representation of the text/entry and hence represents the navigation entry without providing a new action. This applies to For |
@korelstar good points. However even for Calendar, the navigation is very different. There, clicking the navigation entry as well as circle toggles visibility instead of switching to a different group. And in Notes the navigation is a file list. For Files, the file list has the star function on the left of the file. That’s also where it would belong for Notes, even though the list is in the sidebar. :) It’s definitely strange next to the delete button, having the action of »I like this note a lot« and »I hate this note and it’s useless« so close to each other. ;) |
"For notes, the icon has it's own action" |
Maybe, you just haven't any notes marked as favorite. If you mouseover a note in the navigation bar, you see a gray star (currently on the right from the note's title). If you click on that star, the note will be marked as favorite. However, in version 3.0 of the notes app, this behavior will change (see #290 and the discussion under nextcloud-libraries/nextcloud-vue#246) and we will have the star icon on the left side as requested by @jancborchardt |
Well I have some favorites but still no icons on the left of notes titles. I think i may have misunderstand the whole point. Thank you for you answer, you guys seems to be very comitted to this project, it's really nice |
In the current version, 2.5.1, the star isn't on the left side. It's on the right side. From Version 3.0, the star will move to the left side, but toggling the favorite has to be done using a new submenu (three-dot-menu). In all versions, the star indicates if the note is a favorite (yellow) or not (gray). The gray star is only shown, if the note is active or hovered. |
This is how it looks in #290: |
@korelstar very very cool! One tiny thing: If the note is favorited, in the menu it should say "Unfavorite" to reflect what it will do. |
@jancborchardt I've fixed the labels, I'm using the exact wording (and icons) from the files app, now: a78221f |
Currently it’s on the right side. We should put it on the left side to be consistent with the sidebars in other apps.
For that, the
<ul>
just needs theclass="with-icon"
to have padding to the left. Then the star can be positioned on the left of it.cc @korelstar @Henni
The text was updated successfully, but these errors were encountered: