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] website: improve UX of change logo/font size #32291

Closed

Conversation

Projects
None yet
4 participants
@jpr-odoo
Copy link
Contributor

commented Apr 1, 2019

Description of the issue/feature this PR addresses:
Task ID: https://www.odoo.com/web?#id=1952964&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720

Current behavior before PR:

Desired behavior after PR is merged:

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

@robodoo robodoo added the CI 🤖 label Apr 1, 2019

@C3POdoo C3POdoo added the RD label Apr 1, 2019

@jpr-odoo jpr-odoo force-pushed the odoo-dev:master-website-changelogo-size-jpr branch from daabae8 to 45a6f7d Apr 11, 2019

@robodoo robodoo removed the CI 🤖 label Apr 11, 2019

@jpr-odoo jpr-odoo force-pushed the odoo-dev:master-website-changelogo-size-jpr branch from 45a6f7d to fdd92a1 Apr 11, 2019

@robodoo robodoo added the CI 🤖 label Apr 11, 2019

[IMP] website: improve UX for logo/font size changing
This removes the 'V' box and it now validates the entered value when the
user clicks outside the box.

task-1952964

@qsm-odoo qsm-odoo force-pushed the odoo-dev:master-website-changelogo-size-jpr branch from fdd92a1 to 91ae1d6 Apr 16, 2019

@robodoo robodoo removed the CI 🤖 label Apr 16, 2019

@qsm-odoo
Copy link
Contributor

left a comment

Reviewed the commit message to respect conventions :)
@robodoo r+

pniederlag pushed a commit to pniederlag/odoo that referenced this pull request Apr 16, 2019

[IMP] website: improve UX for logo/font size changing
This removes the 'V' box and it now validates the entered value when the
user clicks outside the box.

task-1952964

closes odoo#32291

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>

@robodoo robodoo added merged 🎉 and removed merging 👷 labels Apr 16, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

commented Apr 16, 2019

Merged, thanks!

@robodoo robodoo closed this Apr 16, 2019

@qsm-odoo qsm-odoo deleted the odoo-dev:master-website-changelogo-size-jpr branch Apr 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.