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

feat/tup-486 Create text-wrap for longer links #260

Merged
merged 4 commits into from
Nov 16, 2023

Conversation

R-Tomas-Gonzalez
Copy link
Collaborator

@R-Tomas-Gonzalez R-Tomas-Gonzalez commented Nov 13, 2023

Overview

Adjust card text wrap and icon placement

Related

Changes

  • Adjusts icon down by .2em
  • Allows longer link text to wrap

Testing

Note: When testing, localhost:8000 will use the 1.2em attribute. However, TACC production site uses font-size: 1em/1 cortal.

https://www.tacc.utexas.edu/use-tacc/getting-started/
• Protected Data Services card
• STEM Evaluation Services

UI

Before
Screenshot 2023-11-13 at 1 08 18 PM
After (<a/ > Display Style)
Screenshot 2023-11-13 at 1 07 55 PM
After (<img/ > Position and Top Style)
Screenshot 2023-11-13 at 1 07 32 PM

Notes

Maybe we should delete the font-size or comment it out in core-styles?
*Left it in case this becomes the new norm or this was something that was previously wanted by designers.

@github-actions github-actions bot added the feature A new feature or replacement of existing feature label Nov 13, 2023
@R-Tomas-Gonzalez R-Tomas-Gonzalez changed the title [TUP-486](https://jira.tacc.utexas.edu/browse/TUP-486?filter=14602) feat/tup-486 Create text-wrap for longer links Nov 13, 2023
@R-Tomas-Gonzalez R-Tomas-Gonzalez marked this pull request as ready for review November 13, 2023 19:14
@R-Tomas-Gonzalez R-Tomas-Gonzalez added the minor A feature in backward-compatible manner label Nov 13, 2023
I noticed that top and relative effects other button icons that have middle alignment.
@wesleyboar
Copy link
Member

wesleyboar commented Nov 16, 2023

I noticed the removal of some changes, because icon alignment differs in certain contexts (b46b218). Reasonable decision. Even before this, the alignment differs; so, "fixing icon alignment" can be a separate task, and this PR can be just "fixing wrapping".

@wesleyboar wesleyboar merged commit 7f39284 into main Nov 16, 2023
@wesleyboar wesleyboar deleted the feat/tup-486-card-link-text-wrap branch November 16, 2023 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature or replacement of existing feature minor A feature in backward-compatible manner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants