Skip to content

Commit

Permalink
Fix export label for delimiter #3389
Browse files Browse the repository at this point in the history
  • Loading branch information
Kaspik committed Feb 20, 2019
1 parent 3e40b24 commit c9e1d36
Showing 1 changed file with 40 additions and 52 deletions.

4 comments on commit c9e1d36

@dmoagx
Copy link
Member

@dmoagx dmoagx commented on c9e1d36 Feb 20, 2019

Choose a reason for hiding this comment

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

I'll have to revert this.
When changing IB files please make sure to only commit the changes that were actually intended.

@Kaspik
Copy link
Collaborator Author

@Kaspik Kaspik commented on c9e1d36 Feb 20, 2019

Choose a reason for hiding this comment

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

Okay, does it mean our xibs are still older version for older xcodes?

@dmoagx
Copy link
Member

@dmoagx dmoagx commented on c9e1d36 Feb 20, 2019

Choose a reason for hiding this comment

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

Yes, you can see that right in the header:

<development version="5100" identifier="xcode"/>

5100 means 5.1.

See this table for which OS version that relates to: https://github.com/sequelpro/sequelpro/wiki/Xcode-Compatibility

I personally need them to be compatible to 6.2 and for the next release they must be compatible to 7.3.1 because that is the last version to officially compile for 10.6.

The main issue here however is, that Xcode will change lots of stuff in XIB files every time you just open and save them, even if you didn't make any change (and the next time it will change it back to the previous values, most of the time). That makes it difficult to track the intended changes to the files so our policy is to only commit the actual changes (many git clients allow you to select the changes line by line).

@Kaspik
Copy link
Collaborator Author

@Kaspik Kaspik commented on c9e1d36 Feb 20, 2019

Choose a reason for hiding this comment

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

... yes, I know, one of many reasons why we don’t have yibs/storyboards and do UI in code on our projects. :) thanks for links, I’ll keep an eye on that until next release.

Please sign in to comment.