Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

[UI][mac] Remove background color to show NSAddTemplate #97

Merged
merged 1 commit into from
Nov 16, 2017

Conversation

yamachu
Copy link
Contributor

@yamachu yamachu commented Nov 16, 2017

Remove button color attribute.

As shown in the image below the button is blacked out.
Remove color attribute, button is now displayed.

Before
2017-11-16 9 08 00

After
2017-11-16 9 04 29

@abock
Copy link
Contributor

abock commented Nov 16, 2017

@yamachu thanks for the PR!

A few questions though:

  1. Do you see this issue in one of our released builds?

  2. What version of macOS are you on and what version of Xcode are you using?

@yamachu
Copy link
Contributor Author

yamachu commented Nov 16, 2017

@abock

  1. Yes, I use Xamarin Workbooks 1.4 Beta 1 (master/2e30f47)
    And I feel like I saw this issue in the previous version.
  2. macOS: 10.12.6, Xcode: 9.1 (9B55)

Thanks!

@abock
Copy link
Contributor

abock commented Nov 16, 2017

@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:

@abock
Copy link
Contributor

abock commented Nov 16, 2017

build

@abock
Copy link
Contributor

abock commented Nov 16, 2017

@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
Copy link
Contributor Author

yamachu commented Nov 16, 2017

There is no issue in Xamarin Workbooks 1.3.2 as you mentioned.

2017-11-16 9 45 23

@abock
Copy link
Contributor

abock commented Nov 16, 2017

@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 NSVisualEffectView shines through the button still with backgroundColor removed.

nsvisualeffectview

Copy link
Contributor

@abock abock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 🎉

Copy link
Member

@sandyarmstrong sandyarmstrong left a 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.

Copy link
Contributor

@bojanrajkovic bojanrajkovic left a 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!

Copy link
Member

@lewing lewing left a 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!

@abock abock merged commit df0505c into microsoft:master Nov 16, 2017
@microsoft microsoft locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants