-
Notifications
You must be signed in to change notification settings - Fork 0
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
Show tags in file/folder listing #39
Comments
@marino you can use our e2e.kaminsky.me test server with PROPFIND: PROPFIND with nc:tags |
@tobiasKaminsky but for added / select a tag ? |
Right now it is only for showing tags. |
ok |
@jancborchardt I am unsure if two lines will fit…It will then look on iOS also cluttered, same as on Android… |
Opened nextcloud/desktop#5596 |
Note we will have the same issue on activities on desktop, particularly for those that are already showing changes to several files |
Yes ... isn't good ... |
Yeah, we should definitely not have multiple sublines. How about if there are tags on a file, we just replace the current subline info about file size and date with the tags? We just need to make sure the tag text is on the same baseline as the current date and time text so it looks nice. |
Nice idea. Shall we then still only have one tag? and then "+x" as second? |
We can definitely show more than one tag, however much space there is of course. @tobiasKaminsky for Android the border color of the tag can be a bit lighter, and the tags are a bit too low compared to the size/date text. @marinofaggiana for iOS it looks nice in the file list, good job! Can you there also show more than just 1 tag? 2 at least we could show, of course depending on length of text. :) |
@marinofaggiana @claucambra we now have official API:
For us, right now, only name is important. You can test it/use it on: |
Server:
Webdav property is: "tags".
Output:
Android/iOS @tobiasKaminsky @marinofaggiana:
Desktop @claucambra :
in activity list (ping @nickvergessen if API change in needed), in subline, same as Android/iOSMockup:
The text was updated successfully, but these errors were encountered: