-
Notifications
You must be signed in to change notification settings - Fork 9k
[IMP] website: general themes restructuration #12745
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
Conversation
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.
@guga-odoo thank you for you work on this PR! I think this new structure makes this doc page more clear and overall useful to users. In addition to my review comments:
- Could you please edit your commit message to have the taskid on a new line? 🙏
- I would also document the status colors (like I mentioned in my comment on line 18). Yiu can refer to the last part of these instructions to see what I mean exactly: last part of these instructions: https://scribehow.com/shared/Edit_the_colors__tnLBFBPTQDeCDl6cbfEW-Q)
As usual, let me know if anything’s unclear or if you want to discuss my comments in person!
content/applications/websites/website/web_design/themes/color-presets.png
Outdated
Show resolved
Hide resolved
a5218bd
to
098d70d
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.
Thank you for the updates, @guga-odoo ! Great work adding explanations for the color presets, status colors, and Google Fonts (served from Google servers)! 👏
I’ve added a few more comments and suggestions. Let me know if anything’s unclear or if you’d like to discuss any of them!
098d70d
to
04937d5
Compare
task-4644985
04937d5
to
f600e03
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.
Thank you @guga-odoo for the changes and for working on this! Nice work!
I pushed some minor changes (removed unnecessary spaces/capital letters and added a ref).
@robodoo r+
task-4644985 closes #12745 Signed-off-by: Audrey Vandromme (auva) <[email protected]>
@guga-odoo @auva-odoo staging failed: ci/runbot on 862f689b9b6778cbb24f3e648695adda87812d76 (view more at https://runbot.odoo.com/runbot/build/78110556) |
@robodoo retry |
task-4644985 closes #12745 Signed-off-by: Audrey Vandromme (auva) <[email protected]>
task-4644985