CSS is of part 3

[fringe4life]

Should treat comments like tickets and have mobile actions. Also some skeletons are off

[fringe4life]

Optimistic comments should look different from comments that are actually in the database

[fringe4life]

The actual close button on mobile navigation doesn't work

[fringe4life]

The actual close button on mobile navigation doesn't work

Working in development. Logic seems solid

[fringe4life]

The actual close button on mobile navigation doesn't work

Working in development. Logic seems solid

It only occasionally works. Issue is stacking context, the sheet dialog is higher so if scroll driven animations are supported for now will alter dialog CSS to ensure button is able to close the dialog

[fringe4life]

The actual close button on mobile navigation doesn't work

Working in development. Logic seems solid

It only occasionally works. Issue is stacking context, the sheet dialog is higher so if scroll driven animations are supported for now will alter dialog CSS to ensure button is able to close the dialog

Okay works fine. Now though the dialog overlay dims the header which looks odd

[fringe4life]

Need to fix ticket detail page so that when no comments that the no comments section takes up 1fr. Also need to manage a variety of states, auth/no auth for whether comment form is on the page

[fringe4life]

The actual close button on mobile navigation doesn't work

Working in development. Logic seems solid

It only occasionally works. Issue is stacking context, the sheet dialog is higher so if scroll driven animations are supported for now will alter dialog CSS to ensure button is able to close the dialog

Okay works fine. Now though the dialog overlay dims the header which looks odd

Actually have grown used to this and it doesn't look so bad

[fringe4life]

Overlay for header isn't there when using dialog, noticed this when signing in from interception route

[fringe4life]

Header vanishes after changing page which looks odd

[fringe4life]

Some ticket detail view transitions initially take up more width than they should and then shrink to the correct size. It looks odd. Doesn't happen on safari, but on brave

[fringe4life]

The actual close button on mobile navigation doesn't work

Working in development. Logic seems solid

It only occasionally works. Issue is stacking context, the sheet dialog is higher so if scroll driven animations are supported for now will alter dialog CSS to ensure button is able to close the dialog

Okay works fine. Now though the dialog overlay dims the header which looks odd

Actually have grown used to this and it doesn't look so bad

I was right initially. There are all sorts of edge cases for the way I handled this. The theme and auth button often don't dim at all when mobile menu is open. Really should find a way to make sure only the open close button in the header doesn't dim

[fringe4life]

Okay current solution solves initial issue but created new ones. Current solution for mobile menu button involves taking it out of the header nav entirely and then positioning it fixed. However while this solves the old z index issue and allows it to be above the overlay and therefore clickable it has made it so the scroll driven animation looks off