Skip to content
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

workspace menu is always in hbar-section, fix #480 #482

Merged
merged 2 commits into from
Nov 2, 2024

Conversation

vincent-peugnet
Copy link
Owner

  • remove home specific absolute positioning
  • remove unused "hide in case of js" css

@jbidoret : this is how I would do it.

You tell me if it's ok for you ! 😃

I don't see the problem when only the button is display:none. 🤔 maybe because I did'nt checked all the themes!
Anyway, since with this PR, <form> is always inside a hbar-section, it can be hidden without breaking the flex positioning.

- remove home specific absolute positionning
- remove unused "hide in case of js" css
@vincent-peugnet vincent-peugnet changed the title workspace menu is alwas in hbar-section, fix #480 workspace menu is always in hbar-section, fix #480 Nov 2, 2024
@jbidoret
Copy link
Collaborator

jbidoret commented Nov 2, 2024

I understand the way you solved it, but we still have a little problem with it.

What we have with your PR: (Edit view, js ENabled !)
Capture d’écran du 2024-11-02 14-01-29

What we want :
image

The form, not being completely hidden still takes place in the flexbox, leading to this unwanted space at the end of the hbar. There are multiple ways to solve this issue, but we should think about what might be necessary in the future. IM(H)O, a robust way to transmit to the CSS if js is on or off is necessary, as we may want to adapt the layout and styles from within the CSS, and not in JS.

[ edited : js Enabled, not disabled… ]

@vincent-peugnet
Copy link
Owner Author

vincent-peugnet commented Nov 2, 2024

IM(H)O, a robust way to transmit to the CSS if js is on or off is necessary, as we may want to adapt the layout and styles from within the CSS, and not in JS.

Okkk ! I got it ! (sorry I'm long to understand 😄)

but we still have a little problem with it.

I added a commit that hide the full div instead of just the submit button in order to fix this. (The idea was to fix this issue without needing of changing the JS on/off strategy)

If you're ok with this version as quick fix, I merge this and then you can come with your new workflow (if you want) !

Copy link
Collaborator

@jbidoret jbidoret left a comment

Choose a reason for hiding this comment

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

Cool !

@jbidoret jbidoret merged commit c9b0c1b into master Nov 2, 2024
22 checks passed
@vincent-peugnet vincent-peugnet deleted the workspace-menu-in-hbar-section branch November 2, 2024 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants