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

fix: Break text on post titles so long words don't overflow #1771

Merged

Conversation

jsit
Copy link
Contributor

@jsit jsit commented Jul 3, 2023

Mobile Before

Screenshot 2023-07-02 at 8 22 25 PM

Mobile After

Screenshot 2023-07-02 at 8 19 50 PM

Desktop Before

Screenshot 2023-07-02 at 8 20 41 PM

Desktop After

Screenshot 2023-07-02 at 8 20 59 PM

Link focus ring

This also fixes it so that the link focus ring is not clipped in the title

Before

Screenshot 2023-07-02 at 8 24 22 PM Screenshot 2023-07-02 at 8 24 18 PM

After

Screenshot 2023-07-02 at 8 12 48 PM Screenshot 2023-07-02 at 8 12 45 PM

src/shared/components/post/post-listing.tsx Outdated Show resolved Hide resolved
…titles-overflow

* lemmy/main:
  v0.18.1-rc.9
  fix: Fix comment collapse and vote buttons not having focus style (LemmyNet#1789)
  Add missing modlog reasons (LemmyNet#1787)
  Fix search page breaking on initial load when logged in (LemmyNet#1781)
  feat: Add PR template (LemmyNet#1785)
  v0.18.1-rc.8
  Fix profile loading spinner
  fix: Move getRoleLabelPill to the only component that uses it
  fix: Remove unused hasBadges() function
  fix: Fix badge alignment and break out into component
  fix: Fix up filter row gaps and margins a little
  fix: Fix heading levels
  fix: Simplify row classes a bit
  fix: Fix some gaps in search filters
  fix: Fix row gap on search options
  fix: Add bottom margin to inbox controls
  fix: Small cleanup to search/inbox controls
@SleeplessOne1917 SleeplessOne1917 merged commit 8730f15 into LemmyNet:main Jul 3, 2023
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.

3 participants