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

add support for displaying comment origin instance #944

Merged

Conversation

ggichure
Copy link
Collaborator

@ggichure ggichure commented Dec 2, 2023

Pull Request Description

Add support for displaying comment origin instance. Toggle on and off via settings.

Issue Being Fixed

Add support for displaying comment origin instance
Issue Number: #200

Screenshots / Recordings

Screenshot_1701498378

Checklist

  • Did you update CHANGELOG.md?
  • Did you use localized strings where applicable?
  • Did you add semanticLabels where applicable for accessibility?

Copy link
Member

@micahmo micahmo left a comment

Choose a reason for hiding this comment

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

Hey thanks so much for taking this on! I love the approach you've taken to display the instance name in a slightly different styling than the username. This is something I've liked from Boost/Sync and thought about bringing over here. I wonder if (in the future) we should consider styling community names the same way (instead of with the dot) for consistency. Just a thought!

lib/core/enums/local_settings.dart Outdated Show resolved Hide resolved
lib/shared/comment_header.dart Outdated Show resolved Hide resolved
pubspec.lock Outdated Show resolved Hide resolved
Copy link
Member

@hjiangsu hjiangsu left a comment

Choose a reason for hiding this comment

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

Good job overall, very nice. Just had a few comments!

lib/settings/pages/comment_appearance_settings_page.dart Outdated Show resolved Hide resolved
lib/shared/comment_header.dart Outdated Show resolved Hide resolved
lib/settings/pages/comment_appearance_settings_page.dart Outdated Show resolved Hide resolved
@machinaeZER0
Copy link
Collaborator

This looks great! I like the formatting you used, and (of course) appreciate a good toggle :)

ggichure and others added 2 commits December 8, 2023 08:18
renamed showOriginIntance to commentShowUserInstance,fix typos,fix Richtext text scaling
@hjiangsu hjiangsu merged commit fe2a91c into thunder-app:develop Dec 9, 2023
1 check passed
@micahmo micahmo mentioned this pull request Jan 12, 2024
3 tasks
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