feat(ui/message): support for multi images, collapse long text on messages by JoltCode · Pull Request #393 · AnswerOverflow/AnswerOverflow · GitHub
Skip to content

feat(ui/message): support for multi images, collapse long text on messages#393

Merged
RhysSullivan merged 48 commits into
AnswerOverflow:mainfrom
JoltCode:multi-image-collapse-text
May 19, 2023
Merged

feat(ui/message): support for multi images, collapse long text on messages#393
RhysSullivan merged 48 commits into
AnswerOverflow:mainfrom
JoltCode:multi-image-collapse-text

Conversation

@JoltCode

@JoltCode JoltCode commented May 9, 2023

Copy link
Copy Markdown
Collaborator

Description

  • Support for multi images on a message (grid + modal)
  • Collapses long strings on messages

Fixes #373 (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

N/A

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated .env.example if I added a new environment variable
  • My PR title follows the semantic commits style

@vercel

vercel Bot commented May 9, 2023

Copy link
Copy Markdown

@vercel

vercel Bot commented May 9, 2023

Copy link
Copy Markdown

@RhysSullivan RhysSullivan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

needs collapse content on community page

Comment thread packages/ui/src/components/pages/MessageResultPage.tsx Outdated
Comment thread packages/ui/src/components/pages/SearchPage.stories.tsx Outdated
@RhysSullivan RhysSullivan merged commit 1ad112f into AnswerOverflow:main May 19, 2023
@JoltCode JoltCode deleted the multi-image-collapse-text branch May 19, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚛ component Issue relates to a react component 🖥️website Relates to the website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Web]: Adjust how many images can be shown per post preview

2 participants