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

Try out jenkins-io-components #5668

Merged
merged 1 commit into from Jan 4, 2023

Conversation

halkeye
Copy link
Member

@halkeye halkeye commented Nov 4, 2022

Trying out the web components

  • active links definitely need refactoring.
  • Search box needs to actually be setup

Optional

  • Probably can load it earlier (deferred) so we don't get the flash of a header
  • Probably don't need the polyfills depending on which browers we are okay being degraded and without header/foheader
  • Move GitHub orgs
  • use updatecli instead of always linking to recent package on unpkg?

@halkeye
Copy link
Member Author

halkeye commented Nov 15, 2022

@MarkEWaite @lemeurherve

Getting closer

@halkeye
Copy link
Member Author

halkeye commented Nov 16, 2022

Exciting.

Active links are highlighted nicely now.

Now decisions need to be made.

@halkeye
Copy link
Member Author

halkeye commented Dec 31, 2022

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.

Copy link
Contributor

@MarkEWaite MarkEWaite left a 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.,

@halkeye
Copy link
Member Author

halkeye commented Jan 4, 2023

@halkeye this looks great to me. Are there remaining steps to be taken before it is merged?

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
I have social media icons - https://jenkins-infra.github.io/jenkins-io-components/?path=/story/example-socialmediabuttons--combined so it could be used on author tags
I have the embedded weather icons / voting that could be used too - https://jenkins-infra.github.io/jenkins-io-components/?path=/story/example-changelogratings--all-args

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)

@kmartens27
Copy link
Contributor

everything looks good from my side as well @MarkEWaite & @halkeye, no oddities for a macOS set up that I could find!

@halkeye
Copy link
Member Author

halkeye commented Jan 4, 2023

Oh! I'm no longer on board, I can't merge!

Can someone else merge then.

@kmartens27 kmartens27 merged commit 1dc0ac3 into jenkins-infra:master Jan 4, 2023
@kmartens27
Copy link
Contributor

all set @halkeye!

@NotMyFault
Copy link
Member

I really like the extra attribution for menu topics, but is there a reasoning, why the topmost item isn't highlighted if it's a dropdown list?

Screenshot 2023-01-04 at 22 04 42

@timja
Copy link
Member

timja commented Jan 4, 2023

Blog link on plugins.jenkins.io leads to 404:
https://plugins.jenkins.io/node

@timja
Copy link
Member

timja commented Jan 4, 2023

tbh most links on plugins.jenkins.io 404

@halkeye
Copy link
Member Author

halkeye commented Jan 4, 2023

oops, i have to merge the other two prs now

@halkeye
Copy link
Member Author

halkeye commented Jan 4, 2023

tbh most links on plugins.jenkins.io 404

jenkins-infra/plugin-site#1256 will automerge
jenkins-infra/stories#18 is merged

I really like the extra attribution for menu topics, but is there a reasoning, why the topmost item isn't highlighted if it's a dropdown list?

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"

@halkeye
Copy link
Member Author

halkeye commented Jan 4, 2023

Blog link on plugins.jenkins.io leads to 404:

fixed!

@zbynek
Copy link
Contributor

zbynek commented Jan 5, 2023

wrong links (header and footer) now on https://updates.jenkins.io/

@halkeye
Copy link
Member Author

halkeye commented Jan 5, 2023

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.

@halkeye
Copy link
Member Author

halkeye commented Jan 5, 2023

@NotMyFault
Copy link
Member

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.

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.

None yet

6 participants