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

"Delete Semester" (modal submit) button flicker-shows on staff semester page #2150

Closed
richardebeling opened this issue Mar 25, 2024 · 1 comment · Fixed by #2153
Closed
Labels
[C] Frontend Focuses on frontend implementation [T] Bug This is a bug. We don't like it. Please get rid of it.

Comments

@richardebeling
Copy link
Member

@niklasmohrin we have an anti-flicker css rule in #1985:

confirmation-modal:not(:defined) > :not([slot="submit-group"]) {
// Without this, the elements that are slotted into the dialog element show up as if they are just normal child
// elements and disappear once the custom element registration is done. To avoid a short flicker of these elements,
// we hide them until the constructor of the custom element has run.
display: none;
}

However, the selector here excludes the submit group. For me, on the staff semester index page, the red "Delete Semester" button thus shows for a short period during page load. Removing the > :not([slot="submit-group"]) part of the selector fixes the flickering for me, but I guess there was a reason for this exclusion?

@richardebeling richardebeling added [C] Frontend Focuses on frontend implementation [T] Bug This is a bug. We don't like it. Please get rid of it. labels Mar 25, 2024
@niklasmohrin
Copy link
Member

First impressions: If I remember correctly, I intended to exclude the show-button slot, not the submit-group, not sure how it ended up like this. I can take a closer look later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Frontend Focuses on frontend implementation [T] Bug This is a bug. We don't like it. Please get rid of it.
Development

Successfully merging a pull request may close this issue.

2 participants