-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Try out jenkins-io-components #5668
Conversation
Getting closer |
Exciting. Active links are highlighted nicely now. Now decisions need to be made. |
fa38b73
to
d8b4805
Compare
d8b4805
to
5581c91
Compare
5581c91
to
7db937e
Compare
okay https://deploy-preview-5668--jenkins-io-site-pr.netlify.app/?locale=fr-CA is working (only cdf menu has any translation). The locales files are a bit rough, but i think i have solutions, but thats a later problem. |
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.
@halkeye this looks great to me. Are there remaining steps to be taken before it is merged?
@kmartens27 could you explore the preview site as well in order to see if you see anything amiss. I used Windows and Chrome. Your macOS may show something different.,
nope, nothing blocking. If we like it, i'd like to start converting more things. I have A date time box - https://jenkins-infra.github.io/jenkins-io-components/?path=/story/example-datetimebox--with-start-time-iso that could be reused everwhere instead of just events None of which I want to do before this merged, but the more re-usable components, the easier it will be for future projects/redesigns (web components are entirely self contained, so css issues won't be an issue) |
everything looks good from my side as well @MarkEWaite & @halkeye, no oddities for a macOS set up that I could find! |
7db937e
to
1371abe
Compare
Oh! I'm no longer on board, I can't merge! Can someone else merge then. |
all set @halkeye! |
Blog link on plugins.jenkins.io leads to 404: |
tbh most links on plugins.jenkins.io 404 |
oops, i have to merge the other two prs now |
jenkins-infra/plugin-site#1256 will automerge
time mostly, i found a bunch of that logic a bit more frustrating than I expected, so left it as a straight "if urls match == active", but should be updatable at some point to say "if any active children, then active too" |
fixed! |
wrong links (header and footer) now on https://updates.jenkins.io/ |
I don't know anything about that site's process. Essentially the property attribute for jio-navbar and jio-footer needs to be updated. I kinda feel like with the web components, we probably don't need to be generating that site all the time, it'll just always have the latest nav. |
Do you plan to transition get.jenkins.io from whatever it uses now to the new components too? Looks like that's the last occurrence. |
Trying out the web components
Optional