Skip to content

[18.0][ADD] web_tree_column_keyboard_resize #3412

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.

Sign up for GitHub

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

Jump to bottom
Merged
OCA-git-bot merged 1 commit into from
Jan 15, 2026
Merged

[18.0][ADD] web_tree_column_keyboard_resize #3412

OCA-git-bot merged 1 commit into from
Jan 15, 2026
+610 −0

Conversation

@LauraCForgeFlow
Copy link
Contributor

@LauraCForgeFlow LauraCForgeFlow commented Jan 15, 2026

No description provided.

legalsylvain
legalsylvain requested changes Jan 15, 2026
View reviewed changes
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

tested on chromium and firefox. Don't reach to resize column.
Step to reproduce on runboat;

  • Invoicing > new Customer invoice > click on a new line.
  • click on Shift left / right arrow.

did I missed something ?

thanks !

@LauraCForgeFlow LauraCForgeFlow force-pushed the 18.0-add-web_tree_column_keyboard_resize branch from d4be12a to 6ec372f Compare January 15, 2026 10:27
@LauraCForgeFlow
Copy link
Contributor Author

LauraCForgeFlow commented Jan 15, 2026

Hello @legalsylvain, thanks for checking this out!! Actually, I made a mistake in the documentation of the module, my bad (I already fixed it). The shortcuts are actually the following:

  • Shift + Arrow Up → Increase width
  • Shift + Arrow Down → Decrease width

I've recorded a video to show you how it's supposed to work (testing runboat in Firefox by the way).

web_tree_column_keyboard_resize.mp4

legalsylvain
legalsylvain approved these changes Jan 15, 2026
View reviewed changes
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

functional review. no code review. works like a charm.

LoisRForgeFlow
LoisRForgeFlow approved these changes Jan 15, 2026
View reviewed changes
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@legalsylvain
Copy link
Contributor

legalsylvain commented Jan 15, 2026

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 15, 2026

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-3412-by-legalsylvain-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 892e4fb into OCA:18.0 Jan 15, 2026
9 checks passed
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 15, 2026

Congratulations, your PR was merged at 071ca09. Thanks a lot for contributing to OCA. ❤️

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

Reviewers

@legalsylvain legalsylvain legalsylvain approved these changes

+1 more reviewer

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@LauraCForgeFlow @legalsylvain @OCA-git-bot @LoisRForgeFlow