[19.0][MIG] web_responsive: Migration to 19.0 #3300
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add Tecnativa as author * Remove untestable JS method * Rename to web_responsive
* Bump version * Change all openerp to odoo * Fix qunit suite injection * Remove jQuery require * Change to new selectors: * `oe_leftbar` to `o_sub_menu` * `oe_logo` to `o_sub_menu_logo` * `oe_footer` to `o_sub_menu_footer` * `oe_secondary_menus_container` to `o_sub_menu_content` * Add style to hide oe_footer instead of remove to not break `support_branding` * Add note in ReadMe explaining override of `support_branding` * Set top margin of app drawer title to 0 to fit v10 proportions * Fix notification badge positioning * Add o_web_client class to body to fix overlaps * Scroll control panel with page * Change navbar z-index to not be overlapped by buttons * Raise z-index on header to raise over buttons * Handle layered notifications via z-index * Remove `#` from navigation links HREF to prevent history littering
* Rebalance z-indexes to fix overlay issue while still providing notification support
* Bold titles for apps in the app-drawer * Remove z-index from Control Panel buttons, so it doesn't overlap the menu. * Better responsive menus * Move systray icons/menus out of the menu and up to the top bar. * Increase avatar size and visibility. * Other minor menu behavior fixes. * Changes to fit odoo variables * And a little margin adjust on navbar-right * Top menu always thick * Removed overflow:hidden!important from main.less * Local patch: prevent body overflow auto * Stop using `style=` on `<body>` * Fix jquery drawer in anticipation of: blivesta/drawer#36
On screens with less than 768px content is limited to 2 columns. On mobile field labels have their own line. Fix elements width for XS screens
Make image editing controls always available, instead of depending on resolution or hover.
Small makeup to make it look good in XS screens.
* [IMP] drawer-toggle * [IMP] Hitbox of close drawer-toggle * [IMP] Removed outline on menu item * [IMP] Logo positionings * [FIX] Try me on runbot button * [REM] Reversed outline change
* Change accesskey for `edit` in form view back to `e` to fix OCA#587
Before this patch, there was an incompatibility between this addon and l10n_es_toponyms, caused by the lack of wrapping of the special address field that was being added. Now layout works as expected in that case. In any other case, it seems sensible to wrap inputs so no more weird overflows happen and less chances of needing horizontal scrolling happen.
* Initial commit * Initial commit
* FIX hasclass * FIX views * [FIX] web_responsive: Syntax error in xpath
* Update to iScoll bleeding edge to fix OCA#741
* Add a call to the window resize method during init to fix OCA#773
Currently translated at 29.0% (9 of 31 strings) Translation: web-18.0/web-18.0-web_responsive Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_responsive/fr/
Currently translated at 86.6% (26 of 30 strings) Translation: web-18.0/web-18.0-web_responsive Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_responsive/fr_BE/
Currently translated at 100.0% (31 of 31 strings) Translation: web-18.0/web-18.0-web_responsive Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_responsive/es/
The patch of stepUtils is breaking tests that uses this function when this module is installed. So remove it to avoid them fail.
The selectors used in clickbot.js from the Community Edition do not match any elements in the view when web_responsive is installed. Since the functions are not exported, we can't intercept them directly. To solve this, we override the querySelector functions specifically within the clickbot assets, allowing us to adjust the selectors when they are incompatible with web_responsive.
Currently translated at 96.7% (30 of 31 strings) Translation: web-18.0/web-18.0-web_responsive Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_responsive/tr/
|
You can give us your opinion. |
kobros-tech
suggested changes
Oct 8, 2025
Comment on lines
10
to
12
| <!-- <xpath expr="//SuggestedRecipientsList" position="attributes">--> | ||
| <!-- <attribute name="styleString">''</attribute>--> | ||
| <!-- </xpath>--> |
There was a problem hiding this comment.
if it is useless can be removed or even add the best code for version 19.0?
Comment on lines
10
to
12
| <!-- <xpath expr="//SuggestedRecipientsList" position="attributes">--> | ||
| <!-- <attribute name="styleString">''</attribute>--> | ||
| <!-- </xpath>--> |
There was a problem hiding this comment.
After I reviewed the code, I think the comment can be removed.
d004160 to
ec20678
Compare
|
Several things discovered on runboat tests:
|
ec20678 to
b5f524b
Compare
|
@pedrobaeza : I’ve fixed those issues. Thank you for pointing them out! |
pedrobaeza
approved these changes
Oct 13, 2025
|
ping @kobros-tech |
|
Please check if this PR is necessary in V19 #3323 |
b5f524b to
89bfa02
Compare
The purpose is to reduce spacing so that all icons and buttons fit within the window without having to scroll.
89bfa02 to
593e963
Compare
I’ve done the cherry-pick. Thanks! |
Did you test it after the cherry-pick? I got this error |
|
I also faced the same issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
56 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.