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

feat: Add post body preview to desktop #1455

Merged

Conversation

jsit
Copy link
Contributor

@jsit jsit commented Jun 21, 2023

Added the post body preview to desktop view, and tweaked some post listing alignment

Before

Screenshot 2023-06-21 at 7 42 42 PM Screenshot 2023-06-21 at 7 42 49 PM

After

Screenshot 2023-06-21 at 7 39 23 PM Screenshot 2023-06-21 at 7 39 13 PM

Copy link
Member

@SleeplessOne1917 SleeplessOne1917 left a comment

Choose a reason for hiding this comment

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

Looks good, but I'm holding off on merging until #1456 is merged.

…eview-to-desktop

* origin/main: (40 commits)
  Adding jsit to codeowners.
  Cleanup, only check for /u/ if /c/ and /m/ checks fail
  Rename function to be more generic, since it parses users
  Typescript linter fixes
  bandaid fix our video embeds
  Remove pipe from community link regex
  Add missing classes
  Use shorter regex in community link parser
  Move regex pattern to config
  Update community link markdown parsing
  Fix avatar alignment issue (LemmyNet#1475)
  Omit user-scalable to use default
  Update getHttpBase dependency reference
  Enable users to zoom on mobile
  rethink it a bit
  rethink it a bit
  add fallback style tag
  move env utils into folder
  fix capitalization (LemmyNet#1467)
  Fix buildThemeList() function to ensure no duplicates (LemmyNet#1466)
  ...
@jsit jsit requested a review from alectrocute as a code owner June 22, 2023 21:37
@SleeplessOne1917 SleeplessOne1917 merged commit 7bfa837 into LemmyNet:main Jun 22, 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.

2 participants