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

Ribbon for RC and unsupported #710

Closed
wants to merge 1 commit into from

Conversation

lzap
Copy link
Member

@lzap lzap commented Sep 3, 2021

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

@lzap lzap force-pushed the ribbon-rc-unsupported branch from 74d5ede to e666e2a Compare September 14, 2021 08:46
@lzap
Copy link
Member Author

lzap commented Sep 14, 2021

@ekohl the ribbon named WIP! is now shown for all guides you marked as incorrect, it is clickable and gets you to our README which I elaborated and updated.

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" {
Copy link
Member Author

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.

Copy link
Contributor

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.

@ekohl
Copy link
Member

ekohl commented Sep 14, 2021

@ekohl the ribbon named WIP! is now shown for all guides you marked as incorrect, it is clickable and gets you to our README which I elaborated and updated.

WIP is an abbreviation. Should we use a common English word like Draft?

@lzap lzap force-pushed the ribbon-rc-unsupported branch from e666e2a to 00327f1 Compare September 22, 2021 11:43
@lzap
Copy link
Member Author

lzap commented Sep 22, 2021

Absolutely, draft looks better. Rebased. HTML links will fail, working on that.

@lzap lzap force-pushed the ribbon-rc-unsupported branch from 00327f1 to 09eb8ff Compare September 22, 2021 12:04
@lzap
Copy link
Member Author

lzap commented Oct 20, 2021

@ekohl this is the first step in order to get closer to the ultimate goal, could you re-review please?

@lzap
Copy link
Member Author

lzap commented Nov 11, 2021

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.

Copy link
Contributor

@maximiliankolb maximiliankolb left a 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?

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" {
Copy link
Contributor

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.

@@ -1,8 +1,33 @@
<script>
const rl = document.location.toString();
ifdef::ribbon[]
Copy link
Member Author

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]>
@lzap lzap force-pushed the ribbon-rc-unsupported branch from 09eb8ff to 2725bf9 Compare January 3, 2022 13:05
@lzap
Copy link
Member Author

lzap commented Jan 3, 2022

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.

@lzap lzap closed this Feb 19, 2023
@lzap lzap deleted the ribbon-rc-unsupported branch February 19, 2023 11:30
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 Release Candidate Banner
3 participants