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

Fixed overflow of text in validators on Obsolete Page #597

Merged
merged 6 commits into from
Apr 8, 2024

Conversation

TamannaVerma99
Copy link
Contributor

@TamannaVerma99 TamannaVerma99 commented Mar 24, 2024

What kind of change does this PR introduce?
A bug fix

Issue Number:
#586
Closes #586

Summary
Fixed the text overflow in validators on Obsolete Page by making the required CSS changes.

Does this PR introduce a breaking change?
No

@TamannaVerma99 TamannaVerma99 requested a review from a team as a code owner March 24, 2024 19:02
Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

Great first set of changes. We need to add some more:

  • The column about should be hidden for small devices like we do in the tooling page.
  • I realized that this page was showing the sidebar but it is not required in this case. Please see how the index.page.tsx of the implementations folder works to follow here a similar approach and make sure we don't show the sidebar.

@TamannaVerma99
Copy link
Contributor Author

Hey @benjagm
I am done with all the suggested changes you might have a look at it now.Thanks :)

Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

Great job!

@benjagm benjagm merged commit 1ca6bc5 into json-schema-org:main Apr 8, 2024
2 checks passed
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.

🐛 Bug: Overflow of text in validators on Obsolete Implementations Page.
2 participants