Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Small UI improvements #271

Closed
mejo- opened this issue Dec 19, 2021 · 15 comments
Closed

Small UI improvements #271

mejo- opened this issue Dec 19, 2021 · 15 comments

Comments

@mejo-
Copy link
Member

mejo- commented Dec 19, 2021

In GitLab by @ya-d on Dec 19, 2021, 13:37

I hope, it’s ok to collect some small UI improvements in one issue. Let me know if I should split them up. I apologize for the large screenshots. Keep up the great work and happy holidays! 🎄

This is for Collectives 0.17.22 on NC 23.0.0

  • use print(er) icon for print (probably mdiPrinter or mdiPrinterOutline)
    image

  • Merge App Navigation and options to be one line
    image

  • Sorting options should be consistent with or without arrows. As only ascending sorting is available you could use mdiSortAlphabeticalAscending and mdiSortClockAscendingOutline.
    image

  • The dropdown menu is disconnected from the triangle / arrow. (See above or below.)

  • Template icon seems too big (?)
    image

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit ffdc273035b0f1f8656a0e45f451b9591f6743dd

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit b90c6cbf0d1a067da32adb00b44dffb82f98d906

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit 5d4603faf1fa10270379a950f5536f8f5f472fb8

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit 4aaac994621e80316e8fbf327fb9e0929321d851

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit defbb9ec7539e912eca4c221f18d0e973a4fd062

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit ad3963149dda8e7a80383162a2e146e4c6fd424e

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit 2837c696b1d78f311d481f479fa3e820193360cc

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

Hey @ya-d, thanks for your suggestions. They're highly appreciated.

I opened a merge request with the UI fixes you suggested:

use print(er) icon for print (probably mdiPrinter or mdiPrinterOutline)

https://gitlab.com/collectivecloud/collectives/-/merge_requests/445/diffs?commit_id=ffdc273035b0f1f8656a0e45f451b9591f6743dd

Merge App Navigation and options to be one line

This should already be the case. Which Collectives version to you have installed?

Sorting options should be consistent with or without arrows. As only ascending sorting is available you could use mdiSortAlphabeticalAscending and mdiSortClockAscendingOutline.

https://gitlab.com/collectivecloud/collectives/-/merge_requests/445/diffs?commit_id=5d4603faf1fa10270379a950f5536f8f5f472fb8

The dropdown menu is disconnected from the triangle / arrow. (See above or below.)

https://gitlab.com/collectivecloud/collectives/-/merge_requests/445/diffs?commit_id=2837c696b1d78f311d481f479fa3e820193360cc

Template icon seems too big (?)

https://gitlab.com/collectivecloud/collectives/-/merge_requests/445/diffs?commit_id=4aaac994621e80316e8fbf327fb9e0929321d851

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit c4b25cf

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit 7fe982178a18d932256dcf6941fe0c70990117ed

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit 1816e29

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

mentioned in commit cfbd001

@mejo-
Copy link
Member Author

mejo- commented Dec 20, 2021

In GitLab by @ya-d on Dec 21, 2021, 24:24

Thanks for the quick reply and fixes, Jonas. I’m using Collectives 0.17.22 on NC 23.0.0.

@mejo-
Copy link
Member Author

mejo- commented Dec 21, 2021

mentioned in commit a80b1f5

@mejo-
Copy link
Member Author

mejo- commented Jan 12, 2022

mentioned in commit 965b0de

@mejo- mejo- closed this as completed Dec 13, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

1 participant