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

Updated minimum required Qt versions #2210

Merged
merged 4 commits into from Nov 21, 2020
Merged

Updated minimum required Qt versions #2210

merged 4 commits into from Nov 21, 2020

Conversation

zneix
Copy link
Collaborator

@zneix zneix commented Nov 19, 2020

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

  • Updated minimum required and latest Qt versions
  • Added Qt version check to chatterino.pro

As discused on Chatterino Discord, we now have methods in our code such as QOalette#placeholderText in https://github.com/Chatterino/chatterino2/blob/master/src/widgets/splits/Split.cpp#L321-L322 which were introduced in Qt 5.12.0, updated documentation to reflect those changes.

@zneix
Copy link
Collaborator Author

zneix commented Nov 19, 2020

Darn, I didn't notice #2208 before opening this PR.

@swills
Copy link
Contributor

swills commented Nov 19, 2020

Duplicates #2208 but maybe this one is better

@swills
Copy link
Contributor

swills commented Nov 19, 2020

Yeah, I like this one better, closing #2208

Copy link
Member

@pajlada pajlada 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 to me!

@pajlada pajlada changed the title Updated Qt versions Updated minimum required Qt versions Nov 21, 2020
@pajlada pajlada merged commit 3ffb284 into Chatterino:master Nov 21, 2020
@zneix zneix deleted the updated-qt-versions branch November 21, 2020 12:41
swills pushed a commit to swills/chatterino2 that referenced this pull request Nov 21, 2020
This updates both the build instructions and enforces qt version when running qmake
swills pushed a commit to swills/chatterino2 that referenced this pull request Nov 21, 2020
This updates both the build instructions and enforces qt version when running qmake
@ALazyMeme ALazyMeme mentioned this pull request Aug 4, 2021
1 task
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.

None yet

3 participants