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

registry package pages #1797

Merged
merged 18 commits into from Dec 12, 2022
Merged

registry package pages #1797

merged 18 commits into from Dec 12, 2022

Conversation

sean1588
Copy link
Member

@sean1588 sean1588 commented Dec 7, 2022

@sean1588 sean1588 force-pushed the sean/registry-h1s branch 2 times, most recently from 8d8ccab to 959d9cd Compare December 7, 2022 22:10
@github-actions
Copy link

github-actions bot commented Dec 7, 2022

Your site preview for commit ae2d477 is ready! 🎉

http://registry-origin-pr-1797-ae2d4770.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@pulumi pulumi deleted a comment from github-actions bot Dec 7, 2022
@pulumi pulumi deleted a comment from github-actions bot Dec 7, 2022
@github-actions
Copy link

github-actions bot commented Dec 7, 2022

Your site preview for commit 6a1b92c is ready! 🎉

http://registry-origin-pr-1797-6a1b92c4.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@anita-trimbur
Copy link

anita-trimbur commented Dec 7, 2022

aahhh so exciting!! so glad to see this come together

a few minor style nits for the card itself:

  • the font and size of every element on the open card should be Inter 12px
  • the only font that's different is the "last published" on the collapsed card which is Inter 10px gray600#5A5A5A
  • (this is my bad, it's wrong in figma) we should use blue600 #4D5BD9 for that info icon to match the note containers
  • the info icon and the github icon should be the same size -- i think without the padding it'll be about 15x15
  • i'm pretty sure the border color is a little light on the open card, should be gray300 #DCDCDC. on the closed card it's gray400 #B0B0B0 along with the down/up chevron

and some things from our slack thread:

  • we can drop the font size of the breadcrumbs to 14px on tablet and 12px on mobile
  • we would also like to include the version # on the collapsed card. the format would be v1.13.0, Dec 5 22

@sean1588
Copy link
Member Author

sean1588 commented Dec 7, 2022

awesome. thanks @anita-trimbur. I'll get those changes made!

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit fdd2ef2 is ready! 🎉

http://registry-origin-pr-1797-fdd2ef2e.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit 8ea9e15 is ready! 🎉

http://registry-origin-pr-1797-8ea9e156.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit a71091b is ready! 🎉

http://registry-origin-pr-1797-a71091b0.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit 6ed5db3 is ready! 🎉

http://registry-origin-pr-1797-6ed5db31.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@anita-trimbur
Copy link

anita-trimbur commented Dec 8, 2022

wow you got those changes in fast!

additional nit: can we shrink that external link icon by the source code repo down to 12x12. i think that link text can also be 12px

@anita-trimbur
Copy link

following up on our team discussion regarding grays. here's the breakdown

grays

tailwind is using the blue-grays that we were trying to get rid of. the _colors.scss file nearly matches the service grays in figma with the exception of gray 200, so it shouldn't be problematic to align gray200 as #EFEFEF. i think the difference in border color on the collapsed card was a color sampling error i made in figma -- sean you're right that it should match gray300 as it is defined in the service/.scss file.

@susanev
Copy link
Contributor

susanev commented Dec 8, 2022

@anita-trimbur lets leave 200 as is in the _colors file for now, but get the rest of them updated to match the service ones.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit 3854603 is ready! 🎉

http://registry-origin-pr-1797-3854603b.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit 2398563 is ready! 🎉

http://registry-origin-pr-1797-23985638.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit c644ee3 is ready! 🎉

http://registry-origin-pr-1797-c644ee31.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Your site preview for commit 543462d is ready! 🎉

http://registry-origin-pr-1797-543462d0.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@sean1588 sean1588 marked this pull request as ready for review December 9, 2022 00:45
@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit 45c00b5 is ready! 🎉

http://registry-origin-pr-1797-45c00b57.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit bfa2101 is ready! 🎉

http://registry-origin-pr-1797-bfa2101e.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit 15853fb is ready! 🎉

http://registry-origin-pr-1797-15853fba.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit 67e6598 is ready! 🎉

http://registry-origin-pr-1797-67e65982.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit c28549e is ready! 🎉

http://registry-origin-pr-1797-c28549e2.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Your site preview for commit 8b5e875 is ready! 🎉

http://registry-origin-pr-1797-8b5e8752.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@sean1588 sean1588 requested a review from a team December 12, 2022 16:23
Copy link

@anita-trimbur anita-trimbur left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀

Copy link
Collaborator

@kimberleyamackenzie kimberleyamackenzie left a comment

Choose a reason for hiding this comment

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

Amazing!! 🎉 🚢 :shipit:

@@ -0,0 +1,37 @@
{{ $package := .Scratch.Get "package" }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

TIL about scratch!

Copy link
Member Author

Choose a reason for hiding this comment

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

i just learned about it too 🙂

@github-actions
Copy link

Your site preview for commit bbcb4b7 is ready! 🎉

http://registry-origin-pr-1797-bbcb4b73.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@github-actions
Copy link

Your site preview for commit 72d610b is ready! 🎉

http://registry-origin-pr-1797-72d610b7.s3-website.us-west-2.amazonaws.com/registry.

Registry site previews only include API docs for the AWS and Aiven packages by default. You will not be able to navigate to other docs from the preview link. To see those docs, the PR author can add the desired package name to the list in scripts/ci/build.sh. How-to guides for all packages are also incomplete.

@sean1588 sean1588 merged commit 1122446 into master Dec 12, 2022
@sean1588 sean1588 deleted the sean/registry-h1s branch December 12, 2022 19:12
@github-actions
Copy link

Site previews for this pull request have been removed. ✨

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

4 participants