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

[INFRA] add election banner #1278

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Remi-Gau
Copy link
Collaborator

@Remi-Gau Remi-Gau commented Sep 8, 2022

closes #1276

mkdocs.yml Outdated
features:
- navigation.sections
- header.autohide
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

FYI this will hide a the top banner as people scroll down.

@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #1278 (cbcb892) into master (9ad23e8) will not change coverage.
The diff coverage is n/a.

❗ Current head cbcb892 differs from pull request most recent head dd825d8. Consider uploading reports for the commit dd825d8 to get more accurate results

@@           Coverage Diff           @@
##           master    #1278   +/-   ##
=======================================
  Coverage   88.53%   88.53%           
=======================================
  Files           6        6           
  Lines        1055     1055           
=======================================
  Hits          934      934           
  Misses        121      121           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Remi-Gau
Copy link
Collaborator Author

Remi-Gau commented Sep 8, 2022

This is very rudimentary in terms of stilling. Just to see what people think of the idea.

src/overrides/main.html Outdated Show resolved Hide resolved
Copy link
Member

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

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

LGTM, together with

Co-authored-by: Stefan Appelhoff <[email protected]>
@Remi-Gau
Copy link
Collaborator Author

Remi-Gau commented Sep 8, 2022

LGTM, together with

* setting "dev" as the default page for this short period

that has to be changed on Read the docs, no?

@Remi-Gau
Copy link
Collaborator Author

Remi-Gau commented Sep 9, 2022

Making a couple a aesthetic / readability changes.

Centering a div should be easy, right? (FLW)

@Remi-Gau
Copy link
Collaborator Author

Discussed at maintainers meeting.

Probably better to try to use some Javascript to display a banner on all versions than just a banner on a single one.

@sappelhoff
Copy link
Member

@Remi-Gau I am afraid we let the perfect be the enemy of the good for this one here and now the nomination period is already over 🙈

should we just close this and its corresponding issue? or do you want to use this to implement a mechanism that we can easily switch on/off for the next nomination period?

@Remi-Gau
Copy link
Collaborator Author

leave it open or maybe let's have it as a draft

last meeting it was decided we were just going to javascript the 💩 out of this.

@sappelhoff sappelhoff marked this pull request as draft September 27, 2022 11:03
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.

Add option to toggle announcements on the spec website
2 participants