Skip to content

Conversation

@sorja
Copy link
Contributor

@sorja sorja commented Jan 27, 2026

Screen.Recording.2026-01-27.at.14.04.00.mov

Updated icon color:

Screen.Recording.2026-01-27.at.14.26.18.mov

@sorja sorja changed the base branch from development to 4756-toolbar-area-selector---add-status-filter-updated-sorting January 27, 2026 11:49
@sorja sorja self-assigned this Jan 27, 2026
@sorja sorja changed the title 4756 sorting 4756 - AreaSelector: Allow sorting Jan 27, 2026
@sorja
Copy link
Contributor Author

sorja commented Jan 27, 2026

@minotogna fixed color (new video in desc)

@sorja sorja marked this pull request as ready for review January 27, 2026 12:31
@sorja sorja requested a review from minotogna as a code owner January 27, 2026 12:31
div.select__menu.bottom {
@include min-width($mobile-landscape) {
min-width: 550px;
min-width: 570px;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we customize this only when sorting is available ?


const isAdmin = Users.isAdministrator(user)
const isRegionalFocalPoint = Users.isRegionalFocalPoint(user, countryIso, cycle)
const isReviewer = Users.isReviewer(user, countryIso, cycle)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use isAReviewer and isARegional FocalPoint?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes!!

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.

3 participants