-
Notifications
You must be signed in to change notification settings - Fork 98
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
Ribbon for RC and unsupported #710
Conversation
74d5ede
to
e666e2a
Compare
@ekohl the ribbon named |
guides/common/docinfo.html
Outdated
rl.includes("/Upgrading_and_Updating/index-foreman-el.html" || | ||
rl.includes("/Upgrading_and_Updating/index-foreman-deb.html" || | ||
rl.includes("/Configuring_Load_Balancer/index-foreman-el.html" || | ||
rl.includes("/Configuring_Load_Balancer/index-foreman-deb.html" { |
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.
This could be a regexp, but I specifically want this to be just a list so we can easily be deleting guides which are considered finished.
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 guess this is fine for now IMHO.
WIP is an abbreviation. Should we use a common English word like Draft? |
e666e2a
to
00327f1
Compare
Absolutely, draft looks better. Rebased. HTML links will fail, working on that. |
00327f1
to
09eb8ff
Compare
@ekohl this is the first step in order to get closer to the ultimate goal, could you re-review please? |
If we could land this before 3.1 branching that would be awesome, thanks! Maybe @maximiliankolb could help since @ekohl is busy with getting the bits out. |
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 could not find a version with the "DRAFT" ribbon. Can you please share a link or screenshot?
guides/common/docinfo.html
Outdated
rl.includes("/Upgrading_and_Updating/index-foreman-el.html" || | ||
rl.includes("/Upgrading_and_Updating/index-foreman-deb.html" || | ||
rl.includes("/Configuring_Load_Balancer/index-foreman-el.html" || | ||
rl.includes("/Configuring_Load_Balancer/index-foreman-deb.html" { |
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 guess this is fine for now IMHO.
guides/common/docinfo.html
Outdated
@@ -1,8 +1,33 @@ | |||
<script> | |||
const rl = document.location.toString(); | |||
ifdef::ribbon[] |
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.
This line was breaking JS, not sure what I meant by this. Removing.
Signed-off-by: Lukas Zapletal <[email protected]>
09eb8ff
to
2725bf9
Compare
And there were ton of JS syntax errors, fixed them all now it shows up properly. Apologies for filing untested code. I also updated numbers: 3.1 and 3.0 are supported now, 3.2 will be RC, drafts are from the given list. |
This add ribon "RC" and "Unsupported" according to URL. This is hacky, but it allows us to avoid forking HTML/CSS template. Unfortunately, asciidoc syntax (and attributes) cannot be used in this header "snippet", ideally, this should have been an attribute.
Fixes #695