-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
UI redesign of the menu, location chooser and list pages. #3562
Conversation
OBPIH-4704 Fix dashboard moving when switching sections
OBPIH-4707 Restyling Menu (GSP)
- applied new design styles to location chooser on GSP pages - modified jQuery dialog component - moved logoLabel from header to location chooser button
…lity-gsp OBPIH-4708 & OBPIH-4800 location functionality on gsp
- fix label name typo - add spacing in text - fix all translations to be the same as original
- sort location groups and organizations as last - add translated label for null locations groups and organizations
OBPIH-4707 Fix collapsing menu on GSP for < Admin users
…lity OBPIH-4705 location chooser improvments
OBPIH-4735 Add SearchField component for table filters
…ents and their styles
…aways-to-order-list OBPIH-4924 fix putaway list redirects
…aways-to-order-list OBPIH-4924 bring back putaway list button on show page
OBPIH-4913 Add materialized view for the OrderSummary
OBPIH-4914 Errors with loading / fetching redesigned list pages
OBPIH-4913 Fix quantity ordered calculation on the order item status …
OBPIH-4922 Fix select dropdown on components using utils/Select
@jmiranda, @mdpearson this one is ready (but lets hold with release branch until tomorrow) |
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. I got a quarter of the way through. I can't do any more. We'll fix the broken in the release branch.
@@ -331,7 +331,7 @@ jqueryValidationUi { | |||
|
|||
|
|||
// Allow users to customize logo image url as well as label | |||
openboxes.logo.url = "https://openboxes.com/img/logo_30.png" | |||
openboxes.logo.url = "/openboxes/images/logo/openboxes_logo_40x40.jpg" |
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.
/openboxes should probably be replaced with ${appName}
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.
With that said, we can push that change off until afterwards.
@@ -24,6 +27,40 @@ class ProductApiController extends BaseDomainApiController { | |||
def grailsApplication | |||
def productAvailabilityService | |||
|
|||
def list = { |
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.
We need to remember to include some warnings about this breaking API change.
OBPIH-4914 Combined fixes of issues on list pages
OBPIH-4914 Fix destination/destinationParty when refreshing
No description provided.