Skip to content

[FIX] web: move notifications buttons at the bottom and fix close button - #52984

Closed
IT-Ideas wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-website_notification_close_button_fix2-ltu
Closed

[FIX] web: move notifications buttons at the bottom and fix close button#52984
IT-Ideas wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-website_notification_close_button_fix2-ltu

Conversation

@IT-Ideas

Copy link
Copy Markdown

Prior to this commit, the buttons were place at the right of the notification
message and the close button was misplaced when no title was provided.

After this commit, the buttons are placed at the bottom of the notification so
that the close button is now at the right end of the notification. This commit also
modifies the size of the buttons to btn-sm.

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

@IT-Ideas
IT-Ideas force-pushed the master-website_notification_close_button_fix2-ltu branch 2 times, most recently from 64def94 to a80b3c5 Compare June 15, 2020 15:49
@C3POdoo C3POdoo added the RD research & development, internal work label Jun 15, 2020
Prior to this commit, the buttons were place at the right of the notification
message and the close button was misplaced when no title was provided.

After this commit, the buttons are placed at the bottom of the notification so
that the close button is now at the right end of the notification. This commit also
modifies the size of the buttons to btn-sm.
@IT-Ideas
IT-Ideas force-pushed the master-website_notification_close_button_fix2-ltu branch from a80b3c5 to 91854a1 Compare June 15, 2020 16:02
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jun 15, 2020
@qsm-odoo

Copy link
Copy Markdown
Contributor

Let's go with PR #52982

@qsm-odoo qsm-odoo closed this Jun 17, 2020
@qsm-odoo
qsm-odoo deleted the master-website_notification_close_button_fix2-ltu branch June 17, 2020 06:22
@robodoo robodoo added closed 💔 and removed CI 🤖 Robodoo has seen passing statuses labels Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants