-
-
Notifications
You must be signed in to change notification settings - Fork 619
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
[17.0][ADD] pos_category_vertical_display: New module pos_category_vertical_display #1290
base: 17.0
Are you sure you want to change the base?
Conversation
98ab8c5
to
736c33d
Compare
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.
Hi @peluko00. Thanks for sharing, for such module, could you add a printscreen in the description.md file, to explain what is the change ?
Note :
On runboat, I don't understand what is vertical. Could you elaborate ?
thanks !
Thanks for the feedback @legalsylvain . Basically the current version the scroll of POS categories is horizontally and with this module is vertical. |
736c33d
to
05280e2
Compare
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
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, tested in runboat
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
05280e2
to
ca7dbe2
Compare
ca7dbe2
to
23880d1
Compare
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.
Reviewed code and runbot, LGTM
This module aims to display the POS categories in vertical view when you scroll.
cc https://github.com/APSL 166283
@miquelalzanillas @lbarry-apsl @javierobcn @mpascuall @BernatObrador @ppyczko please review
Old version:
New version: