-
Notifications
You must be signed in to change notification settings - Fork 642
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
NuGet survey banner #8249
NuGet survey banner #8249
Conversation
968587e
to
55383fd
Compare
</span> | ||
</div> | ||
</div> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd consider moving this to be the last banner, after the cookie and warning banners. I'm guessing the cookie banner needs to be the first thing on the page, is that correct @zhhyu?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please double-check with @zhhyu about banner ordering before merging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Answer comment then
Cookie banner should be on the top. I will move the survey banner after cookie and warning |
55383fd
to
9eae6c8
Compare
This reverts commit 53729e8.
UI:
Addresses #8225