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

Qoldev 77 hyperlinks on dfv #986

Merged
merged 3 commits into from
Jun 20, 2023

Conversation

ghazal-salehi
Copy link
Contributor

@ghazal-salehi ghazal-salehi commented Jun 19, 2023

Applying button styles to Dfv portal
This includes default, active, hover states. Also the vertical spacing in the link list is fixed as well.

https://oss-uat.clients.squiz.net/community/getting-support-health-social-issue/support-victims-abuse/domestic-family-violence/i-want-to-help-someone

Before:
image

After:
image

Before:
image

After:
image

@ghazal-salehi ghazal-salehi self-assigned this Jun 19, 2023
}
#qg-primary-content .dfv-page-introduction a, #qg-primary-content .dfv-content a:not(.qg-links-list__link) {
color: $qg-blue-dark !important;
}

#qg-content #qg-primary-content .dfv-topics__link {
margin: 0 !important;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does this need to be !important?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is to override the default 2px margin. I can replace it with @include qg-global-link-styles($margin: 0);
I'll update it soon.

@ghazal-salehi ghazal-salehi requested a review from ThrawnCA June 20, 2023 01:38
@ThrawnCA
Copy link
Contributor

Can the build failures be looked into before approving and merging this? They might not be caused by this change, but I'd like to fix them before proceeding with confidence.

@ghazal-salehi
Copy link
Contributor Author

Can the build failures be looked into before approving and merging this? They might not be caused by this change, but I'd like to fix them before proceeding with confidence.

It's the footer feedback test that Steve fixed yesterday. It was merged this morning. I'll rebase my changes to make sure.

@ghazal-salehi
Copy link
Contributor Author

Buiid was successful.

@ghazal-salehi ghazal-salehi merged commit 6244d57 into release-v4.3.0 Jun 20, 2023
@duttonw duttonw deleted the QOLDEV-77-hyperlinks-on-DFV branch July 17, 2023 03:07
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.

2 participants