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 light theme to Qdarkstyle #268

Merged
merged 13 commits into from
Mar 29, 2021
Merged

Conversation

juanis2112
Copy link
Contributor

@juanis2112 juanis2112 commented Mar 23, 2021

This PR adds everything needed for having a light theme in QDarkstyle

Screenshot 2021-03-23 at 3 54 28 PM

Screenshot 2021-03-23 at 3 54 38 PM

Screenshot 2021-03-23 at 3 54 49 PM

Screenshot 2021-03-23 at 3 55 02 PM

Screenshot 2021-03-23 at 3 55 22 PM

@juanis2112
Copy link
Contributor Author

I rebased this with develop in order to include the last merged changes. I think this one is ready for review @dpizetta

@ccordoba12 ccordoba12 added this to the 3.0 milestone Mar 28, 2021
@ccordoba12
Copy link
Collaborator

@dpizetta, I merged again with develop to fix the merge conflicts with your latest changes.

@dpizetta dpizetta mentioned this pull request Mar 29, 2021
@dpizetta dpizetta merged commit 52cb661 into ColinDuquesnoy:develop Mar 29, 2021
@juanis2112
Copy link
Contributor Author

Hi Daniel!

I'm posting here a screenshot of the light theme in Spyder and things are working fine! (The cantered tabs in editor and console will be fixed in Spyder but they are not related to QDarkstyle).
Could you please release 3.0 so @ccordoba12 can make the release tomorrow?

Screenshot 2021-03-31 at 5 53 07 PM

@dpizetta
Copy link
Collaborator

dpizetta commented Apr 1, 2021

@juanis2112 tks, it is already released :) Please check the buttons I mentioned in #260.

Tks for your collaboration!

@ccordoba12
Copy link
Collaborator

Thanks a lot for your help @dpizetta! It's been great collaborating with you on getting this release done!

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.

3 participants