Skip to content

[19.0][IMP] web_responsive: sticky search panel on mobile - #3748

Open
CarlosRoca13 wants to merge 1 commit into
OCA:19.0from
Tecnativa:19.0-IMP-web_responsive-sticky-panel
Open

CarlosRoca13 wants to merge 1 commit into
OCA:19.0from
Tecnativa:19.0-IMP-web_responsive-sticky-panel

Conversation

@CarlosRoca13

Copy link
Copy Markdown
Contributor

On small screens, the bar that replaces the search panel sidebar now stays sticky at the top while scrolling. This applies to every view with a search panel, such as the product catalog.

When the control panel is visible, the bar sits right below it. The control panel now exposes its visible height in the --ControlPanel-sticky-offset CSS variable, so the bar can follow it as the control panel hides and shows.

The content container now grows with the view instead of keeping the screen height. Otherwise the bar would stop being sticky after scrolling one screen down.

This is a FW-PORT of #3743

ping @adasatorres-tecnativa @pedrobaeza

On small screens, the bar that replaces the search panel sidebar now stays sticky at the top while scrolling. This applies to every view with a search panel, such as the product catalog.

When the control panel is visible, the bar sits right below it. The control panel now exposes its visible height in the --ControlPanel-sticky-offset CSS variable, so the bar can follow it as the control panel hides and shows.

The content container now grows with the view instead of keeping the screen height. Otherwise the bar would stop being sticky after scrolling one screen down.
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @Tardo, @SplashS,
some modules you are maintaining are being modified, check this out!

@pedrobaeza

Copy link
Copy Markdown
Member

The sticky property is not the same as the systray:

Peek 21-10-2026 11-53

Is it worth to also hide it by default as the systray?

@CarlosRoca13

Copy link
Copy Markdown
Contributor Author

The sticky property is not the same as the systray:

Peek 21-10-2026 11-53 Is it worth to also hide it by default as the systray?

In my opinion I think it should be always visible as it is when we are in the PC... but if you think that it should be hided like the systray, I can implement it

@pedrobaeza

Copy link
Copy Markdown
Member

Well, it's a space saver, but accessible just scrolling down a bit if needed. Take into account that on desktop, you have more space, and it's affordable to keep it unfolded. This one is an improvement, as it appears folded, but even that, it's using some precious vertical space if you don't plan to use it, so using the same hiding technique is worth IMO.

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.

4 participants