Skip to content

WEB-63 Format blog page to display style #350

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jun 11, 2023
Merged

WEB-63 Format blog page to display style #350

merged 5 commits into from
Jun 11, 2023

Conversation

winnteas
Copy link
Contributor

Why the changes are required?

  • blog page currently does not display the style of content as it does in the editor

Changes

  • fixed blog overflow issue for styled text
  • added min-width in text to properly display the alignments
  • used the same properties from editor page for code blocks + quote blocks
  • also added fixes for the URL of blog page + unhandled runtime error popup

Example:
blog id: ea545375-a2f6-40a8-b55f-bd20da6d10de
image

image

lauraw0
lauraw0 approved these changes May 14, 2023
View reviewed changes
Copy link
Contributor

@lauraw0 lauraw0 left a comment

Choose a reason for hiding this comment

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

lgtm! merge whenever

@winnteas winnteas merged commit 3c8d63f into main Jun 11, 2023
@winnteas winnteas deleted the WEB63-display-content-style branch June 11, 2023 12:30
@winnteas winnteas restored the WEB63-display-content-style branch June 11, 2023 12:30
@winnteas winnteas deleted the WEB63-display-content-style branch June 11, 2023 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lauraw0 lauraw0 lauraw0 approved these changes

@jamest0 jamest0 Awaiting requested review from jamest0

@hdphuong hdphuong Awaiting requested review from hdphuong

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@winnteas @lauraw0