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

Remove titleAccessoryView #removed #935

Merged
merged 2 commits into from
Mar 4, 2021
Merged

Remove titleAccessoryView #removed #935

merged 2 commits into from
Mar 4, 2021

Conversation

Kaspik
Copy link
Member

@Kaspik Kaspik commented Mar 3, 2021

Changes:

  • Remove titleaccessoryView as it's not used anyways anymore

Closes following issues:

  • Closes: Nothing

Tested:

  • Processors:
    • Intel
    • Apple Silicon
  • macOS Versions:
    • 10.12.x (Sierra)
    • 10.13.x (High Sierra)
    • 10.14.x (Mojave)
    • 10.15.x (Catalina)
    • 11.x (Big Sur)
  • Xcode Version:
  • Localizations:
    • English
    • Spanish
    • Other (please specify)

Screenshots:

Additional notes:

I have no idea what this view was supposed to do but I can't see it anywhere and I don't see a difference when removing it. It might improve #857

@Kaspik Kaspik changed the title Remove titleaccessoryView #removed Remove titleAccessoryView #removed Mar 3, 2021
Copy link
Member

@Jason-Morcos Jason-Morcos left a comment

Choose a reason for hiding this comment

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

Looks okay, are we certain we're not using this to show the "SSH Connected" stuff?
Let's hold until after the 3.2.3 release, just so we can test to make sure no funky edge cases before pushing. I'm pushing 3.2.3 to apple now, but let's hold until it's actually approved and out (I'm a little concerned @jamesstout's awesome "check for update" change will cause us to get rejected and we may have to disable it in the App Store version somehow.)

@Kaspik
Copy link
Member Author

Kaspik commented Mar 3, 2021

I have no idea, but iirc that was funky-broken too, right? 😂 Yeah I'm okay with merging tomorrow.

@Jason-Morcos
Copy link
Member

I have no idea, but iirc that was funky-broken too, right? 😂 Yeah I'm okay with merging tomorrow.

Screen Shot 2021-03-03 at 14 45 00

A little funky but still mostly working lol

@Kaspik
Copy link
Member Author

Kaspik commented Mar 3, 2021

Why would we be hiding it on full screen, and why do we have image there (in the xib) for this? 🤷🏻‍♂️

@Jason-Morcos
Copy link
Member

Why would we be hiding it on full screen, and why do we have image there (in the xib) for this? 🤷🏻‍♂️

Good point, looks like it indeed is just some silly legacy thing. Good to remove, next release :)

@Kaspik
Copy link
Member Author

Kaspik commented Mar 4, 2021

3.2.3 is out 🎉

@Kaspik Kaspik merged commit 479f8a8 into main Mar 4, 2021
@Kaspik Kaspik deleted the titleAccessory branch March 4, 2021 10:53
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.

None yet

2 participants