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

feat(toolbar): Make the login-success page have styles that blend with the login flow #81230

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

ryan953
Copy link
Member

@ryan953 ryan953 commented Nov 25, 2024

With timeout (?delay=3000) Debug Mode (?delay=0)
SCR-20241125-pabk SCR-20241125-nxmx

@ryan953 ryan953 requested a review from a team November 25, 2024 09:07
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 25, 2024
Comment on lines -53 to -56
{% comment %}
No need to close `body`. If we do then middleware will inject some extra markup
we don't need. Browsers can figure out when it missing and deal with it.
{% endcomment %}
Copy link
Member

Choose a reason for hiding this comment

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

Does the middleware get injected again now?

Copy link
Member Author

@ryan953 ryan953 Nov 26, 2024

Choose a reason for hiding this comment

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

ya, there's loads of default stuff that's brought in now, perf timing... i wasted a moment last week fidgeting with the footer i guess.

I think though that this template is done-done at this point!

@ryan953 ryan953 merged commit a727fc6 into master Nov 26, 2024
49 checks passed
@ryan953 ryan953 deleted the ryan953/toolbar-success-html-styles branch November 26, 2024 01:13
andrewshie-sentry pushed a commit that referenced this pull request Dec 2, 2024
…h the login flow (#81230)

| With timeout (`?delay=3000`) | Debug Mode (`?delay=0`) |
| --- | --- |
| <img width="802" alt="SCR-20241125-pabk"
src="https://github.com/user-attachments/assets/8af9b71b-229e-46e2-b542-6215a37e7a84">
| <img width="802" alt="SCR-20241125-nxmx"
src="https://github.com/user-attachments/assets/ddc71216-563c-474e-b5c8-4016224e0649">
@github-actions github-actions bot locked and limited conversation to collaborators Dec 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants