Skip to content

UI picker refresh#8036

Draft
gevorgmansuryan wants to merge 4 commits into
masterfrom
enh/picker-refresh
Draft

UI picker refresh#8036
gevorgmansuryan wants to merge 4 commits into
masterfrom
enh/picker-refresh

Conversation

@gevorgmansuryan
Copy link
Copy Markdown
Contributor

@gevorgmansuryan gevorgmansuryan commented Feb 25, 2026

@gevorgmansuryan gevorgmansuryan marked this pull request as draft February 25, 2026 23:28
…resh

# Conflicts:
#	static/js/humhub/humhub.ui.picker.js
@gevorgmansuryan
Copy link
Copy Markdown
Contributor Author

@mbumpalumpa can you please review?

@mbumpalumpa mbumpalumpa requested review from mbumpalumpa and removed request for mbumpalumpa April 6, 2026 21:57
@mbumpalumpa mbumpalumpa self-assigned this Apr 6, 2026
@mbumpalumpa
Copy link
Copy Markdown
Contributor

@gevorgmansuryan there are some issues and one change.

So you can see in the screenshots that the alignment isn't the best. People/Space images are outside of the pill itself, and in the case of Post Type, you can see that the icon is sticking to the bottom rather than being centered. Also, you can see that the pill position in the picker itself looks like the bottom padding/margin is larger.

Additionally, the change would be the X within the pill itself. The original concept indeed used the "X" that you implemented; however, the Font Awesome X is way too thick and dominates the pill itself, so we should use fa-times-circle within the pills.

Additionally, I would expect pills and pickers to look the same regardless of where they are. You can see that the User Picker Pill in Stream Filter and the User Picker Pill in Group Members look different, aka the pill is larger, which is good. Basically, we currently have two different sizes/heights: 23px and 28px. I would much rather standardize this to 28px across the board.

Screen Shot 2026-04-20 at 11 49 22 Screen Shot 2026-04-26 at 20 20 12

Here is an example of pickers for Design System v2; you can see that all spacing is unified (Ignore the design itself, which is a lot different than what we currently have).

Screen Shot 2026-04-26 at 20 50 47

@gevorgmansuryan
Copy link
Copy Markdown
Contributor Author

@mbumpalumpa thanks! Should be fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants