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

Refine Paragraph Styles Scope in the Authorize Application Screen #8673

Closed
4 tasks done
hussain-t opened this issue May 8, 2024 · 4 comments
Closed
4 tasks done

Refine Paragraph Styles Scope in the Authorize Application Screen #8673

hussain-t opened this issue May 8, 2024 · 4 comments
Labels
P2 Low priority Squad 2 (Team M) Issues for Squad 2 Type: Enhancement Improvement of an existing feature

Comments

@hussain-t
Copy link
Collaborator

hussain-t commented May 8, 2024

Feature Description

The current implementation of paragraph styles on the Authorize Application screen applies too broadly, affecting all paragraphs, including those outside the intended .auth-app-card area. This issue aims to refine the scope of these styles to ensure they only affect the relevant sections of the screen, improving overall style specificity and reducing potential style conflicts.

image (1)


Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

  • Paragraph Style Scope: The paragraph styles currently applied globally under .authorize-application-php should be restricted to only affect elements within the .auth-app-card section.
  • Custom Footer Paragraph Styling: The paragraph styling specifics should also be applied appropriately to the paragraph within the custom footer section.

Implementation Brief

In assets/sass/authorize-application/_googlesitekit-authorize-application.scss:

  • Move the existing paragraph style rules under .authorize-application-php to ensure they only affect elements within the .auth-app-card section. See:
    p {
    font-size: $fs-authorize-application-label-lg;
    font-weight: $fw-medium;
    line-height: $lh-authorize-application-label-lg;
    margin: 19px 0 20px;
    }
  • Apply the following specific paragraph styles to the .googlesitekit-authorize-application__footer p element:
    • line-height: $lh-authorize-application-label-lg
    • margin: 19px 0 20px

Test Coverage

  • No new tests are required for this change.

QA Brief

  • Navigate to the following URL on your WordPress site, replacing example.com with your actual site domain:
https://example.com/wp-admin/authorize-application.php?app_name=GoogleServiceIntegration&app_id=123e4567-e89b-12d3-a456-426614174000&success_url=https%3A%2F%2Fexample.google.com%2Fsettings%2Fauthorization%2Fwordpress&sitekit=true
  • Verify that the paragraph styles are only applied to elements within the .auth-app-card section.
  • Verify that the custom footer paragraph styling is correctly applied to the paragraph within the custom footer section.
  • Verify that the paragraph styles are not applied to any other screen sections.

Changelog entry

  • Improve style specificity in the Authorize Application screen.
@hussain-t hussain-t self-assigned this May 8, 2024
@hussain-t hussain-t added P2 Low priority Type: Enhancement Improvement of an existing feature Squad 2 (Team M) Issues for Squad 2 labels May 8, 2024
@hussain-t hussain-t removed their assignment May 8, 2024
@ivonac4 ivonac4 added the Next Up Issues to prioritize for definition label May 13, 2024
@techanvil techanvil self-assigned this May 13, 2024
@techanvil
Copy link
Collaborator

IB ✅, thanks @hussain-t!

@techanvil techanvil removed their assignment May 13, 2024
@hussain-t hussain-t self-assigned this May 14, 2024
@ivonac4 ivonac4 removed the Next Up Issues to prioritize for definition label May 14, 2024
@hussain-t hussain-t removed their assignment May 15, 2024
@nfmohit nfmohit self-assigned this May 27, 2024
nfmohit added a commit that referenced this issue May 27, 2024
@nfmohit nfmohit removed their assignment May 27, 2024
@kelvinballoo kelvinballoo self-assigned this May 28, 2024
@kelvinballoo
Copy link
Collaborator

QA Update ⚠️

Hi @hussain-t , did a test on this and it's looking good overall. I do have some clarifications to be confirmed.

Observations are as follows:

  • Most of the Paragraph elements are within the .auth-app-cardsection. ✅

    Screenshot 2024-05-28 at 18 08 36
  • The other footer paragraph are under wpfooter and googlesitekit-authorize-application__footer respectively. I believe that's as expected but can you confirm. ⚠️

    Screenshot 2024-05-28 at 18 23 13
  • I noticed other paragraph elements on the page under different sections wp-auth-fallback , even though those are not visible. Any concern for those? ⚠️

    Screenshot 2024-05-28 at 18 27 56

@hussain-t
Copy link
Collaborator Author

Hi @kelvinballoo, regarding your observations:

The other footer paragraph are under wpfooter and googlesitekit-authorize-application__footer respectively. I believe that's as expected but can you confirm.

That's expected.

I noticed other paragraph elements on the page under different sections wp-auth-fallback , even though those are not visible. Any concern for those?

We only want to apply the nested paragraph styles to the auth-app-card and footer sections.

@kelvinballoo
Copy link
Collaborator

kelvinballoo commented May 28, 2024

QA update ✅

Thanks @hussain-t
Moving this to 'Approval' queue.

Most of the Paragraph elements are within the .auth-app-cardsection. ✅

Screenshot 2024-05-28 at 18 08 36

The other footer paragraph are under wpfooter and googlesitekit-authorize-application__footer respectively, which is as expected. ✅

Screenshot 2024-05-28 at 18 23 13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Low priority Squad 2 (Team M) Issues for Squad 2 Type: Enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

6 participants