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

USWDS-Site - About: Move release notes section to What’s New #2318

Merged
merged 34 commits into from
Feb 27, 2024

Conversation

mahoneycm
Copy link
Contributor

@mahoneycm mahoneycm commented Oct 25, 2023

Summary

Moved release notes section into the what’s new page and adds a note about subscribing to releases on GitHub. This PR also removes the two column layout from the What’s New page to accommodate for the new section.

Related issue

Closes #2136

Preview link

What’s new →

Problem statement

The current Release Notes page improperly displays release notes that are pulled in from GitHub.

Solution

Since this is a low traffic page, we’ve opted to move release notes information to the What’s New page and directly link out to GitHub release notes.

From our Office Hours meeting notes (Google Docs 🔒):

Conclusion: Remove release notes page. Forward the release notes url to “what’s new” page. Add a release notes section to the what’s new page. In that section, add link to release notes in github and info about subscribing to releases.

Testing and review

  1. Visit What’s New page.
  2. Confirm content appropriately informs users of where to find release notes as well as how to subscribe to releases.
  3. Check grammar, punctuation, and spelling.
  4. Approve of page layout.
  5. Inspect community page.
  6. Confirm external link buttons have launch icon and appropriate margins.
  7. Confirm “Release notes” page is removed from site navigation.
  8. Confirm there are no links to the release notes page.

Testing checklist

  • Information on release notes is properly conveyed.
  • Information on how to subscribe to releases is clear and easy to follow.
  • What’s New page is well presented.
  • Prose descendant external links styled as buttons have external link icon.
  • Standard external links unaffected by this change.
  • Internal links styled as buttons unaffected by this change.
  • No grammatical or spelling errors.
  • Expected redirects are present and working properly.
  • HTMLProofer runs without error.

Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

  • Confirmed that the releases page was removed from the site
  • Confirmed that redirects have been set up and links to the releases page have been updated
  • Confirmed that the new content makes sense and is free from error
    • I added some notes for possible improvements. I think it would also be good if @finekatie or @sarah-sch could take a look to see if this is the right copy for the page. I've tagged you both as reviewers in case you are able to take a look!

pages/whats-new/overview.md Show resolved Hide resolved
_security_updates/2021-10-27-santized-combo-box-content.md Outdated Show resolved Hide resolved
pages/whats-new/overview.md Outdated Show resolved Hide resolved
pages/whats-new/overview.md Outdated Show resolved Hide resolved
pages/whats-new/overview.md Show resolved Hide resolved
pages/documentation/implementations.md Show resolved Hide resolved
pages/whats-new/overview.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

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

Resolved remaining references to whats-new/releases, corrected release notes links, and updated copy for new Releases section.

Thanks for the recommendations @amyleadem @finekatie

@mahoneycm
Copy link
Contributor Author

External link button

I realized that since the button is an external link, it should include the external link class and launch icon.

The icon is not displaying currently and I'm working to resolve it now.

Current:

Screenshot 2023-11-06 at 10 58 30 AM

Expected behavior:
image

@mahoneycm
Copy link
Contributor Author

External link button cont.

It looks like we're overriding the external link styles for buttons on site:

.usa-button,
.link-download,
.media__link {
&#{$external-href}::after {
display: none;
}
}

If we remove usa-button from that style definition it displays properly.

image

Can you think of a reason we wouldn't want the external link icon on external links styled as buttons?

@amyleadem @mejiaj

@amyleadem
Copy link
Contributor

amyleadem commented Nov 7, 2023

@mahoneycm I like the idea of including the external link icon when usa-button points to an href outside our domain. I did a quick trial of just removing the .usa-button from that style rule (line 1218) and immediately noticed that the github icon in the nav header received an external link icon. I'm wondering if we should either narrow the "display:none" rule to just where it is needed or do an override just for something like usa-buttons inside of usa-prose.

cc: @mejiaj

@mahoneycm
Copy link
Contributor Author

External link buttons styles

Ok I added some styles specific to external links styled as buttons which are descendants of usa-prose (Thanks for the idea @amyleadem !)

Additionally, I skimmed through parts of the site for other external links styled as buttons and noticed some visual discrepancies brought on by utility classes. With these additional styles I was able to safely remove the utility classes 👍

I also added the site-button class to match other site buttons.

Screenshots

Community page before:

image

Community page after:

image

Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

Looking good! Just a couple of small things for your consideration.

pages/about/community.md Outdated Show resolved Hide resolved
css/_uswds-theme-custom-styles.scss Outdated Show resolved Hide resolved
@amyleadem
Copy link
Contributor

Realized that this PR does not have a changelog. Will work on adding one now.

@amyleadem
Copy link
Contributor

Update: Added a changelog. Also re-instated the in-page nav on the page now that the content is longer and laid out in a more vertical format.

Copy link
Contributor

@mejiaj mejiaj left a comment

Choose a reason for hiding this comment

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

Minor comment on external links check. Also found this quirk when clicking on Sidenav What's new:

image

How to reproduce

  1. Visit Preview →
  2. In sidenav, confirm two missing sections Latest Updates and Empty link.

Copy link
Contributor

@sarah-sch sarah-sch left a comment

Choose a reason for hiding this comment

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

Suggesting updated language for first paragraph under "Release notes."

{% capture blog %}
## Release notes

You can find summaries of bug fixes, new features, and other updates introduced in each release on our [GitHub releases page](https://github.com/uswds/uswds/releases).
Copy link
Contributor

Choose a reason for hiding this comment

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

New suggestion for first paragraph, but interested to know if @finekatie has any tweaks:

Find summaries of each USWDS update on our GitHub releases page. Release notes explain bug fixes, new features, and other changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in 11a0ab7

@finekatie
Copy link
Contributor

Information on release notes is properly conveyed.
Information on how to subscribe to releases is clear and easy to follow.
No grammatical or spelling errors.

I do notice that the first CTA goes to live link. The other two go to federalist links. Is that because the changes/updates are being made to the Release Notes section only?

@mahoneycm
Copy link
Contributor Author

Minor comment on external links check. Also found this quirk when clicking on Sidenav What's new:

image **How to reproduce**
  1. Visit Preview →
  2. In sidenav, confirm two missing sections Latest Updates and Empty link.

@mejiaj this seems to have been resolved with Amy's fix in cffa8d3

@mahoneycm
Copy link
Contributor Author

@finekatie @sarah-sch Ready for re-review!

And to answer your question Katie

I do notice that the first CTA goes to live link. The other two go to federalist links. Is that because the changes/updates are being made to the Release Notes section only?

The two Federalist links are due to them being internal links. Since the new CTA is to GitHub, it'll actually take you to the live site 👍

Copy link
Contributor

@finekatie finekatie left a comment

Choose a reason for hiding this comment

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

Charlie answered my question about CTA links, so I have no other input. Ok from here.

Copy link
Contributor

@sarah-sch sarah-sch left a comment

Choose a reason for hiding this comment

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

Approving changes--looks good.

Copy link
Member

@thisisdano thisisdano left a comment

Choose a reason for hiding this comment

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

A nice improvement

@thisisdano thisisdano merged commit ddd921b into main Feb 27, 2024
11 checks passed
@thisisdano thisisdano deleted the cm-release-notes-update branch February 27, 2024 00:23
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.

USWDS-Site - Bug: Release Notes not rendering
6 participants