-
Notifications
You must be signed in to change notification settings - Fork 221
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
(feat) O3-4362: Move Location Picker to the top nav #1295
base: main
Are you sure you want to change the base?
Conversation
23f8f91
to
0aaf0b8
Compare
Size Change: +981 B (+0.02%) Total Size: 6.21 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @chibongho !
|
||
.divider { | ||
width: 1px; | ||
height: 24px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
height: 24px; | |
height: layout.$spacing-06; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Nicely done, @chibongho!
Requirements
feat
,fix
, orchore
, among others). See existing PR titles for inspiration.For changes to apps
If applicable
Summary
This PR adds the Location Picker directly onto the top nav, and removes it from the top-right user menu. Based on mockup here.
Minor deviations from mockup:
$layout.spacingXX
size.Screenshots
ltr, desktop:

ltr, mobile:

rtl, desktop:

rtl, mobile:

Related Issue
https://openmrs.atlassian.net/browse/O3-4362
Other