-
Notifications
You must be signed in to change notification settings - Fork 79
[UI][mac] Remove background color to show NSAddTemplate #97
[UI][mac] Remove background color to show NSAddTemplate #97
Conversation
@yamachu thanks for the PR! A few questions though:
|
Thanks! |
@yamachu thanks. We're all on the same macOS and Xcode as you as well. Would you mind double checking a 1.3.2 install? For 1.4.x we moved the build to Xcode 9 which introduced a number of CALayer-related issues. I have never seen this issue before, so I'd love to verify a released build that was built against Xcode 8.3.x: |
build |
@yamachu you will actually need to fully uninstall your 1.4.0-beta1 install before installing 1.3.2: sudo /Library/Frameworks/Xamarin.Interactive.framework/Versions/Current/uninstall https://developer.xamarin.com/guides/cross-platform/workbooks/install/#Downgrading |
@yamachu excellent - that confirms it being an issue with Xcode 9.1. Thank you! I just tested your build from VSTS and confirm that the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Awesome, glad we don't need this anymore. It was part of a hack to emulate NSOutlineView
's sidebar
style without being layer-backed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this!
Remove button color attribute.
As shown in the image below the button is blacked out.
Remove color attribute, button is now displayed.
Before

After
