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 colours for dark mode #2639

Merged
merged 6 commits into from
Feb 17, 2022
Merged

Fix colours for dark mode #2639

merged 6 commits into from
Feb 17, 2022

Conversation

pjrobertson
Copy link
Member

Removes previous 'custom' colours (yellow/purple etc.) from Triggers/Catalog/Plugins views, but by not hard-coding colours, we can use the system's defaults

Fixes #2638

Removes previous 'custom' colours (yellow/purple etc.) from Triggers/Catalog/Plugins views, but by not hard-coding colours, we can use the system's defaults
@pjrobertson pjrobertson added this to the 2.0.0 milestone Feb 16, 2022
@pjrobertson
Copy link
Member Author

Some screenshots (could still be improved, but this is much better than before).
Screenshot 2022-02-16 at 13 38 14
Screenshot 2022-02-16 at 13 38 17
Screenshot 2022-02-16 at 13 38 23

@n8henrie
Copy link
Member

Looks great here too, thanks for sorting this out!

#2537 (comment)

@pjrobertson
Copy link
Member Author

pjrobertson commented Feb 16, 2022

@n8henrie @skurfer - there's one more view that I can't update. Xcode keeps crashing on me. I'm only on Xcode 11, maybe it was fixed in a later Xcode. Can you try the following:

  1. Open up the QSHelpersPrefPane.xib file
  2. Accept the warning:
    Screenshot 2022-02-16 at 18 44 36
  3. Click on the table, then in the sidebar drill down until you get to the 'table view' item (see left of pic below)
  4. After clicking the table, go to the right sidebar and change 'Grid Color' to 'Separator Color'
  5. Save, commit, then push back here.

Screenshot 2022-02-16 at 18 47 43

Every time I hit save, Xcode crashes :(

@skurfer
Copy link
Member

skurfer commented Feb 16, 2022

there's one more view that I can't update. Xcode keeps crashing on me.

That’s been happening since Xcode 8. I upgraded all the NIBs at one point, but had to skip that one. I even tried using the command-line ibtool on it, but no luck.

We talked about it in #2295. This other comment makes it sound like I fixed it somehow, but obviously, I didn’t.

@pjrobertson
Copy link
Member Author

OK, if that's the case I'll follow the instructions here

The crash log for Xcode gives some explanation:

ASSERTION FAILURE in /Library/Caches/com.apple.xbs/Sources/IDEInterfaceBuilder/IDEInterfaceBuilder-15505/InterfaceBuilderKit/Document/IBDocument.m:2557
Details:  An instance of NSWindowTemplate with object ID 5 did not archive its child (NSPopUpButtonCell) with an object of ID 144.
Edit log: (
    "Change Grid Color"
)

@pjrobertson
Copy link
Member Author

@skurfer - I did see one mention that it was an issue with Mojave. If possible, can you please try opening and saving on your latest Xcode/macOs, just in case. If it's still no-go I'll follow the steps above

Also: disable 'Continue' button when scanning catalog/getting plugins. Fixes #2525
@skurfer
Copy link
Member

skurfer commented Feb 16, 2022

Opening and saving in Xcode 13.2 on macOS 12.2.1 → Instant crash

@pjrobertson
Copy link
Member Author

👍 for checking, thanks. 👎 for Xcode! I'll look into it.

@n8henrie
Copy link
Member

My saga with the insta-crash on QSHelpersPrefPane.xib (and a few posts below): #2537 (comment)

I couldn't get it figured out. Tried looking at the diffs made with the Xcode "auto-fix" on similar files to see if I could do it manually, but it didn't seem practical.

@skurfer
Copy link
Member

skurfer commented Feb 16, 2022

Wait, just like I said in the linked comment, removing the two button cells at the bottom allows me to save it. Do they do anything? Trying to test now.

@pjrobertson
Copy link
Member Author

Rob to the rescue! That's it, great catch. Those NSPopupCells were just 'floating' which probably isn't allowed and hence why Xcode was crashing.

I was able to open and delete them as well. Xib updated and pushed here. This PR should be good to go now.

@skurfer skurfer merged commit 196525e into master Feb 17, 2022
@skurfer skurfer deleted the darkmode branch February 17, 2022 13:24
skurfer added a commit that referenced this pull request Feb 17, 2022
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.

Preferences look ugly in Dark Mode
3 participants