Skip to content

[16.0][ADD] web_form_view_resizable - #2976

Open
sbejaoui wants to merge 1 commit into
OCA:16.0from
acsone:16.0-web_form_view_resizable-sbj
Open

sbejaoui wants to merge 1 commit into
OCA:16.0from
acsone:16.0-web_form_view_resizable-sbj

Conversation

@sbejaoui

@sbejaoui sbejaoui commented Oct 29, 2024

Copy link
Copy Markdown
Contributor

This module allows users to resize the form view, enhancing usability by providing additional space for better display. It's especially practical when more room is needed to view attachments in the chatter or to expand the form view for improved readability.

web_form_view_resizable (1)

@legalsylvain

Copy link
Copy Markdown
Contributor

Hi @sbejaoui. Thanks for sharing this module.

I tried it on runboat, and it doesn't work on my computer (with Firefox & Chromium). Is it working on runboat for you ?

thanks !

System : Ubuntu 20.04
Firefox : 131.0.2 (64 bits)
Chromium : Version 130.0.6723.58 (Build officiel) snap (64 bits)

@sbejaoui

Copy link
Copy Markdown
Contributor Author

Hi @sbejaoui. Thanks for sharing this module.

I tried it on runboat, and it doesn't work on my computer (with Firefox & Chromium). Is it working on runboat for you ?

thanks !

System : Ubuntu 20.04 Firefox : 131.0.2 (64 bits) Chromium : Version 130.0.6723.58 (Build officiel) snap (64 bits)

Yes, it's working for me on runboat. Try the database with only the base module; I believe there may be a conflict with another module.

@tarteo tarteo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sbejaoui I've tested it on runbot on the baseonly version it works there. Only a minor problem is that when I grab the resize handle it shoots to the right a couple of pixels first(like 100px or so). Is it possible to make the minWidth and maxWidth dependable on screen size?

It conflicts with web_responsive, is it possible to make it work with existing modules?

@sbejaoui

sbejaoui commented Nov 7, 2024

Copy link
Copy Markdown
Contributor Author

@sbejaoui I've tested it on runbot on the baseonly version it works there. Only a minor problem is that when I grab the resize handle it shoots to the right a couple of pixels first(like 100px or so). Is it possible to make the minWidth and maxWidth dependable on screen size?

It conflicts with web_responsive, is it possible to make it work with existing modules?

Thank you for the debug, I will make the changes

@pedrobaeza

Copy link
Copy Markdown
Member

Thanks for this! I have tried and it seems that max/min size of the elements is limited:

Peek 07-11-2024 19-55

@SAnnabelle SAnnabelle left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionally it's good.

@javierizaca javierizaca left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review OK LGTM

This module allows users to resize the form view, enhancing usability by providing
additional space for better display. It's especially practical when more room is needed
to view attachments in the chatter or to expand the form view for improved readability.
@sbejaoui
sbejaoui force-pushed the 16.0-web_form_view_resizable-sbj branch from 68ce1ff to 1a0f7b0 Compare June 22, 2025 12:36
captivea-JohnTouba added a commit to Riss-Group/web that referenced this pull request Jun 18, 2026
… jamais mergée)

patch() API 17 : 2 args (sans nom) + super.setup() (ex-this._super). Pour cdd_all.
@petrus-v

Copy link
Copy Markdown

@sbejaoui if I properly understand the history, when you have rebase in june you have drop the source code suggested by acsone#18 that were present when some people approved this PR

Could you go back to the 68ce1ff commit (and rebase if needs) or tell us why you remove it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants