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

BUG: Fixing width on Topic Collection Pages #7442

Merged
merged 9 commits into from
Apr 2, 2024

Conversation

clmedders
Copy link
Contributor

@clmedders clmedders commented Mar 26, 2024

Summary

Adding a width of 100% to fix issue with cards not going full width.

Preview

Link to Preview

Solution

Adding width: 100%; to address the error that was according when text was below a certain amount. This was causing the width to be dependent on the about the text that was present on the card.

How To Test

  1. Visit the following links and compare live site to fix (also visit DG BUG: width & alignment in topic collection pages for more context)
Live Preview
https://digital.gov/topics/content-strategy/ Fix - Content Strategy
https://digital.gov/topics/search/ Fix - Search
https://digital.gov/topics/usability/ Fix - Usability

Dev Checklist

  • PR has correct labels
  • A11y testing (voice over testing, meets WCAG, run axe tools)
  • Code is formatted properly

Copy link

🔍 Preview in Federalist

@clmedders clmedders marked this pull request as ready for review March 26, 2024 19:27
@@ -51,13 +51,6 @@ <h2 class="card-event__title" id="{{ .Title | urlize }}">
</div>
{{- end -}}

{{- if .Params.host -}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

after reviewing the file this code was getting a display none and looked to not being used at all.

@@ -242,10 +243,6 @@
}
}

.card-event__host {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removing due too display none and took out code above

@clmedders clmedders requested review from nick-mon1 and mejiaj March 26, 2024 20:00
@clmedders clmedders self-assigned this Mar 26, 2024
@clmedders clmedders added the Dev: frontend ideas and issues related to the presentation layer of the site label Mar 26, 2024
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.

LGTM, nice find on the host display.

Added an additional note as to why this is happening and requested a small change. Looks like branch is outdated too.

themes/digital.gov/src/scss/new/_card-event.scss Outdated Show resolved Hide resolved
@mejiaj mejiaj self-requested a review March 29, 2024 16:16
Copy link
Contributor

@nick-mon1 nick-mon1 left a comment

Choose a reason for hiding this comment

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

I tested the 3 topic pages links, looks good!

@ToniBonittoGSA ToniBonittoGSA merged commit 9f20208 into main Apr 2, 2024
4 checks passed
@ToniBonittoGSA ToniBonittoGSA deleted the cm-bug-collection-page branch April 2, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev: frontend ideas and issues related to the presentation layer of the site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants