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: dialog height #162

Merged
merged 1 commit into from
Mar 29, 2023
Merged

fix: dialog height #162

merged 1 commit into from
Mar 29, 2023

Conversation

himicoswilson
Copy link
Contributor

@himicoswilson himicoswilson commented Mar 29, 2023

目前來看其他沒有被影響到
CleanShot 2023-03-29 at 19 52 22@2x

@vercel
Copy link

vercel bot commented Mar 29, 2023

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

@Yidadaa first needs to authorize it.

@Yidadaa
Copy link
Collaborator

Yidadaa commented Mar 29, 2023

看起来是 #148 引入的,可以把这个 pr 的 line-break 去掉看看。如果单纯去掉 margin-bottom,可能会让多段落排版出问题。

@himicoswilson
Copy link
Contributor Author

line-break 去掉後可以解決這個問題

@Yidadaa Yidadaa merged commit 73b2ede into ChatGPTNextWeb:main Mar 29, 2023
@RugerMcCarthy
Copy link
Contributor

去掉之后ReactMarkdown中将不展示换行

yxl pushed a commit to yxl/ChatGPT-Next-Web that referenced this pull request May 4, 2023
ryiann pushed a commit to ryiann/ChatGPT that referenced this pull request Dec 1, 2023
- [+] fix(settings.tsx): fix conditional rendering of ListItem based on clientConfig.isApp
- [+] refactor(settings.tsx): improve readability of conditional rendering code
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.

4 participants