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

Paints support for chatterino7 #86

Closed
wants to merge 3 commits into from
Closed

Paints support for chatterino7 #86

wants to merge 3 commits into from

Conversation

williambulin
Copy link

Pull request checklist:

  • Color gradients
  • Angle rotation
  • Username color blending
  • Shadows
  • API calls

Platforms tested:

  • Windows
  • macOS
  • Linux (tested with GCC 11.2.0 on arch x64 btw)

Screenshots

image

Notes

  • There's no obvious way to do shadows with Qt that I could find for the text rendering method Chatterino uses.
  • We still need to do an API call to associate a user with a paint.
  • Paints are also functionnal to mentionned users in chat right now.

Additional notes

  • SevenTVGradient objects are meant to be cached as shown in the example, however you cannot cache the QPen objects since they are generated with the user color for blending (you could cache the ones that don't require color blending however, or cache it per user or per chat color).
  • Rendering the text twice with the gradient on top to blend with the original username unfortunately looks terrible and therefore isn't really an option, on top of having potential performance concerns.

@TroyKomodo
Copy link

Any news on this PR?

@LosFarmosCTL LosFarmosCTL mentioned this pull request May 15, 2022
11 tasks
@AnatoleAM
Copy link

This was picked up at #89

@AnatoleAM AnatoleAM closed this May 15, 2022
JAB-dev pushed a commit to JAB-dev/chatterino7WithDirtyFileUpload that referenced this pull request Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants