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

[SD-551]AV-heading-not-visible #562

Merged
merged 3 commits into from
Dec 18, 2024

Conversation

yeniatencio
Copy link
Contributor

@yeniatencio yeniatencio commented Dec 11, 2024

Jira

https://digital-vic.atlassian.net/browse/SD-551
BE link: https://nginx-php.pr-1756.content-vic.sdp4.sdp.vic.gov.au/

Problem/Motivation

Heading for AV is not fully visible:
image-20241203-031756

Fix

Fixed styling by adding text-wrap property.

Related PRs

Screenshots

Screenshot 2024-12-11 at 12 03 04 pm

TODO

Copy link
Contributor

@MdNadimHossain MdNadimHossain left a comment

Choose a reason for hiding this comment

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

@yeniatencio small change, please look into the comment

@@ -33,6 +33,7 @@
box-shadow: none !important;
border: none !important;
padding: 0 0 0.5em;
text-wrap: auto;
Copy link
Contributor

Choose a reason for hiding this comment

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

@yeniatencio auto does not actually count as a valid value anymore and it is not a global value as well. In this case you should use a valid value for text-wrap it self. In this case use text-wrap: wrap; or text-wrap: balance; or text-wrap: pretty; or text-wrap: stable;

/* Keyword values */
text-wrap: wrap;
text-wrap: nowrap;
text-wrap: balance;
text-wrap: pretty;
text-wrap: stable;

/* Global values */
text-wrap: inherit;
text-wrap: initial;
text-wrap: revert;
text-wrap: revert-layer;
text-wrap: unset;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @MdNadimHossain . I have updated my PR.

@yeniatencio yeniatencio merged commit c870823 into develop Dec 18, 2024
1 check passed
@yeniatencio yeniatencio deleted the feature/SD-551-AV-heading-not-visible-fully branch December 18, 2024 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants