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

Align RichText heading styles and font sizes with OOB SharePoint text web part #1706

Conversation

mbice
Copy link
Contributor

@mbice mbice commented Nov 21, 2023

Q A
Bug fix? [x]
New feature? [ ]
New sample? [ ]
Related issues? fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?

The out-of-the-box SharePoint text web part has changed over the past year and looks different than the RichText component in several ways. This PR aligns the RichText component so it's more close to the out-of-the-box text web part. The following alignments were made.

  • Update styles dropdown strings for headings (e.g. instead of Heading 1 to 3 it's now Heading 2 to 4). Also moved the "Normal text" option below the headings to match with SharePoint text web part.
  • Update font sizes to match out-of-the-box. This includes the font size options in the dropdown AND the font sizes applied in the CSS. Headings and text in the RichText component should now match SharePoint text web part.
  • Update pull quote styling to match SharePoint text web part.

Thanks!

@mbice mbice changed the title Align heading styles and font sizes with OOB SharePoint text web part Align RichText heading styles and font sizes with OOB SharePoint text web part Nov 21, 2023
@joelfmrodrigues joelfmrodrigues merged commit b7806cf into pnp:dev Jan 30, 2024
1 check passed
@joelfmrodrigues
Copy link
Collaborator

@mbice many thanks and sorry for the delay, this is now merged

@joelfmrodrigues joelfmrodrigues added this to the 3.17.0 milestone Jan 30, 2024
@joelfmrodrigues joelfmrodrigues mentioned this pull request Feb 5, 2024
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.

None yet

2 participants