setup-your-mac / Setup-Your-Mac

Setup Your Mac aims to simplify initial device configuration by leveraging swiftDialog and Jamf Pro Policy Custom Events to allow end-users to self-complete Mac setup post-enrollment.
https://snelson.us/sym
MIT License
231 stars 52 forks source link

brandingBannerDisplayText Not Respected #139

Closed seaneldridge7 closed 1 month ago

seaneldridge7 commented 4 months ago

Before submitting a Setup Your Mac bug report, please review the open swiftDialog issues to help determine the source of the issue.

Describe the bug Despite having brandingBannerDisplayText="false" the text still appears over the top of a custom branding banner.

To Reproduce

Expected behavior I wouldn't expect to see anything other than the custom branding bannner

Code/log output brandingBanner="/Users/Shared/Resources/Wallpapers/custom.png" brandingBannerDisplayText="false"

Screenshots image

Environment (please complete the following information):

Additional context This problem manifested itself at the same time as the update to swiftDialog 2.4.1

drtaru commented 4 months ago

Hi @seaneldridge7, Are you using the WelcomeDialog to collect userInput? If yes, does the banner display correctly there, with no text?

seaneldridge7 commented 4 months ago

Hi @seaneldridge7, Are you using the WelcomeDialog to collect userInput? If yes, does the banner display correctly there, with no text?

Hi @drtaru, no we're not using that in our workflow. However, if I take a fresh copy of the script and only modify those two lines with a custom banner and set the text to false, the same problem occurs at the userInput screen.

Screenshot 2024-02-14 at 2 26 55 PM

drtaru commented 4 months ago

Perfect, thanks, that helps me narrow it down. I will dig in!

drtaru commented 4 months ago

Found the issue, seems like if bannerText is set to nothing "" then the title parameter takes over The fix is to set bannerText=" " instead of bannerText="" when brandingBannerDisplayText="false" PR #140 should fix this, you can wait for it to be merged or test the changes yourself now.

dan-snelson commented 1 month ago

This should have been closed back in February.