-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[ADD][web_editor_background_color] Custom background color picker #582
Conversation
This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those.
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.
code review: 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.
Hey @yajo
I tested it on runbot:
- I tried to select a color in the color picker (by clicking on the color pallet): The background wouldn't change to the color I selected; it kept changing it to white. It works though when I select the text field, type in any character (the menu closes and the background color changes to black), reopen the menu and then select the color on the pallet.
- The color presets works.
Code:
- Looks very good. It's easy to read and has some nice inline documentation!
I'm using Google Chrome 56.0.2924.87.
Thanks @tarteo, could you upload a screencast to see exactly what's happening? |
@yajo |
This happened specifically in Chrome as demonstrated in OCA#582 (comment).
@tarteo It should work now. I did not notice because I mainly use Firefox. Merger: remember to squash on merge 😊 |
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.
👍 😄 👍
var style = this.$target.prop("style"); | ||
this.change_border = | ||
style["border-color"] && | ||
style["background-color"] == style["border-color"]; |
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.
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.
Tested in runbot
@moylop260 Fixed. 4x 👍 now. Please merge 🙌 😊 |
…A#582) * [ADD][web_editor_background_color] Custom background color picker This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those. * [FIX] Avoid clicks removing dropdown menu This happened specifically in Chrome as demonstrated in OCA#582 (comment). * [FIX] Use strict equality operator.
…A#582) * [ADD][web_editor_background_color] Custom background color picker This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those. * [FIX] Avoid clicks removing dropdown menu This happened specifically in Chrome as demonstrated in OCA#582 (comment). * [FIX] Use strict equality operator.
…A#582) * [ADD][web_editor_background_color] Custom background color picker This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those. * [FIX] Avoid clicks removing dropdown menu This happened specifically in Chrome as demonstrated in OCA#582 (comment). * [FIX] Use strict equality operator.
…A#582) * [ADD][web_editor_background_color] Custom background color picker This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those. * [FIX] Avoid clicks removing dropdown menu This happened specifically in Chrome as demonstrated in OCA#582 (comment). * [FIX] Use strict equality operator.
…A#582) * [ADD][web_editor_background_color] Custom background color picker This addon lets the user choose a preferred background color when editing some web content. Works out of the box with `website` and `mass_mailing` addons, although requires none of those. * [FIX] Avoid clicks removing dropdown menu This happened specifically in Chrome as demonstrated in OCA#582 (comment). * [FIX] Use strict equality operator.
This addon lets the user choose a preferred background color when editing some web content.
Works out of the box with
website
andmass_mailing
addons, although requires none of those.😎 Cool
This addon replaces the old
website_mail_snippet_bg_color
, and it is completely new.@Tecnativa