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

Blog Title Overflow Fix #578

Merged
merged 3 commits into from
Sep 23, 2022
Merged

Conversation

stephaniePocci
Copy link
Collaborator

This PR fixes issue #274

I was able to fix the blog listings so that there is no more text overflow from the titles. The titles now wrap, truncate, and place ellipsis. See screenshots below:
image
image
I'm unable to show the changes in vercel since I had to enable debug mode for localhost changes, so reviewers may need to clone my fork locally to view the changes. Please feel free to let me know though if there is a way to enable the blogs to be viewed in vercel!

Also, I noticed there is another issue where the titles still do text overflow on the actual blog pages, so I will be making an issue for that soon.

@vercel
Copy link

vercel bot commented Sep 21, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @EthanThatOneKid on Vercel.

@EthanThatOneKid first needs to authorize it.

@vercel
Copy link

vercel bot commented Sep 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
acm-csuf-site ✅ Ready (Inspect) Visit Preview Sep 23, 2022 at 6:08AM (UTC)

src/routes/blog/index.svelte Outdated Show resolved Hide resolved
src/routes/blog/index.svelte Outdated Show resolved Hide resolved
@jaasonw jaasonw linked an issue Sep 22, 2022 that may be closed by this pull request
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@jaasonw jaasonw left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@karnikaavelumani karnikaavelumani left a comment

Choose a reason for hiding this comment

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

So proud of you! Great work:)

Copy link
Contributor

@charliettaylor charliettaylor left a comment

Choose a reason for hiding this comment

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

May the flood of text not overflow the blog (Psalm 69:15)

Copy link
Collaborator

@anguzz anguzz left a comment

Choose a reason for hiding this comment

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

2 good /ᐠ。ꞈ。ᐟ\

@stephaniePocci stephaniePocci merged commit 8bba956 into EthanThatOneKid:main Sep 23, 2022
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.

title overflow
6 participants