Skip to content
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

[IMP]base: added horizontal scroll for onboarding bars #30823

Conversation

pka-odoo
Copy link
Contributor

@pka-odoo pka-odoo commented Feb 5, 2019

  • improve onboarding bars design on mobile: Use horizontal scroll

task: 1929048

Description of the issue/feature this PR addresses:
Replace the vertical scroll on the onboarding bars by a horizontal scroll to gain space on small devices

Current behavior before PR:
The design of the onboarding widget on the application like Invoicing or Website is taking a lot of space on mobile.

Desired behavior after PR is merged:
Replace the vertical scroll on the onboarding bars by a horizontal scroll to gain space on small devices

Task:
https://www.odoo.com/web#id=1929048&action=327&model=project.task&view_type=form&menu_id=4720

Pad:
https://pad.odoo.com/p/r.604964ed2b5912583e9d8f63429dec47

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@C3POdoo C3POdoo added the RD research & development, internal work label Feb 5, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 5, 2019
@dharmrajsinh-jhala dharmrajsinh-jhala force-pushed the master-mobile-improve-onboarding-bars-pka branch from f6c34d5 to 8cef662 Compare February 11, 2019 08:40
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 11, 2019
@dharmrajsinh-jhala
Copy link
Contributor

LGTM

@pka-odoo pka-odoo force-pushed the master-mobile-improve-onboarding-bars-pka branch from 8cef662 to 35a5983 Compare February 18, 2019 12:33
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 18, 2019
@pka-odoo pka-odoo force-pushed the master-mobile-improve-onboarding-bars-pka branch from 35a5983 to cc572c5 Compare February 19, 2019 09:59
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Feb 19, 2019
@pka-odoo pka-odoo force-pushed the master-mobile-improve-onboarding-bars-pka branch 2 times, most recently from ec43a75 to d7eadbd Compare February 19, 2019 11:39
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 19, 2019
In small devices, onboarding bars are displayed vertial, which
takes too much screen space, and user has to scroll down
a lot to view actual content.

This commit improves the behavior by showing onboarding bars
horizontally (with scrolling) in small devices also, and thus
occupying minimal screen space.

task: 1929048
@adr-odoo adr-odoo force-pushed the master-mobile-improve-onboarding-bars-pka branch from d7eadbd to 4538416 Compare February 22, 2019 13:20
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Feb 22, 2019
@adr-odoo
Copy link
Contributor

@robodoo r+

@robodoo robodoo added r+ 👌 CI 🤖 Robodoo has seen passing statuses labels Feb 22, 2019
robodoo pushed a commit that referenced this pull request Feb 22, 2019
In small devices, onboarding bars are displayed vertial, which
takes too much screen space, and user has to scroll down
a lot to view actual content.

This commit improves the behavior by showing onboarding bars
horizontally (with scrolling) in small devices also, and thus
occupying minimal screen space.

task: 1929048

closes #30823
@robodoo
Copy link
Contributor

robodoo commented Feb 22, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 22, 2019
@adr-odoo adr-odoo deleted the master-mobile-improve-onboarding-bars-pka branch February 25, 2019 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants