aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* +[NSThread detachNewThreadWithName:…] changesMax2014-12-301-4/+56
| | | | | The method never did the one extra thing it should do, because… see comment in the code. Let's see how this works out :)
* Fix two minor CSV import dialog issuesMax2014-12-261-22/+24
| | | | | * Tooltips were somewhat broken for non-imported fields (issue exists in 1.0.2) * Newly added expressions would cause the field to switch to "do not import" instead of the added expression
* ************** IMPORTANT: SEE BELOW **************Max2014-12-263-195/+268
| | | | | | | | | This change should fix the "slow CSV import dialog" issue on 10.10 (details below). To accomplish that, some major changes had to be made to the CSV import code. PLEASE VERIFY THE CORRECTNESS OF THE NEXT FEW CSV IMPORTS YOU DO! * Renamed some variables for better search visibility * Changed some instances where NSNumbers were being compared using "==" instead of "isEqual…" (does not work with object literals!) * The CSV import dialog would recreate the popup menus on *every* call to "tableView:objectValueFor…". However Instruments suggests that [NSMenu removeAllItems] and [NSMenu addItemsWithTitles:] are **really** expensive, esp. when called multiple times per second (e.g. when scrolling in the table view). This commit moves the popup menu setup into its own method and only calls it when the data actually changes. * The 'destination table column' was for some reason a NSPopupButtonCell which only ever had one item and sometimes got changed to a NSTextFieldCell in code. I didn't see any need to keep the popup cell and replaced it with a simple text cell.
* Add some more stub codeMax2014-12-233-1/+16
| | | | (as the nib loader on 10.10 is even whinier)
* Fix two threading issuesMax2014-12-231-25/+29
| | | | | * A call of setString: from background thread which could cause a "deleted thread with uncommitted CATransaction" warning * SPFieldMapperController was allocated from a background thread with an own autoreleasepool, while it contains UI objects. This could cause a crash when the autoreleasepool was drained and the field mapper forcefully released, while other UI objects still had it retained.
* Change CSV field mapper table highlight styleMax2014-12-231-1/+0
| | | | (Purely guessing here…)
* Add GUI option for utf8mb4 connection encodingMax2014-12-192-4/+115
|
* Touch all XIB files with Xcode 6.1.1Max2014-12-1928-1317/+27176
| | | | | No"real" changes were made. NOTE: There is actually a chance for this change to break stuff.
* Add code for utf8mb4 encodingMax2014-12-192-19/+22
|
* Roll some duplicate code into a loopMax2014-12-141-59/+33
| | | | (so the compiler can unroll it again)
* Fix an exception when opening Bundle EditorMax2014-12-141-4/+3
| | | | Caused by an earlier commit of mine.
* Fix crash on export with "overwrite existing files"Max2014-12-141-3/+10
| | | | This should fix #2010. A required object was not retained long enough.
* Attempt 2 to fix building on 10.7, 10.8Max2014-12-135-18/+16
|
* Attempt 1 to fix building on 10.7Max2014-12-132-0/+10
|
* Fix issue #2040Max2014-12-131-0/+6
| | | | Changing the query favorties/history would cause an UI inconsitency when the list was filtered before.
* Replace some NSDictionaries with literalsMax2014-12-1348-465/+437
| | | | | | * [NSDictionary dictionary] → @{} * [NSDictionary dictionaryWithObject:forKey:] can safely be replaced. object==nil would have already thrown a NPE in the past. * Also replaced some (hopefully safe) NSArray initializers (ie. their objects should always exist).
* Change [NSArray arrayWithObject:] to @[] literalMax2014-12-1325-61/+60
| | | | Note: [NSArray arrayWithObjects:...,nil] is left unchanged as that could possibly cause a NPE if converted to @[]
* Add cast for [NSApp delegate]Max2014-12-1331-100/+108
| | | | | * (Also changed some outdated URLs) * Replaced [NSApplication sharedApplication] with NSApp
* Replace boxed integers in code with @ literalsMax2014-12-1318-65/+65
|
* Reformatting code for modern ObjCMax2014-12-1325-135/+132
| | | | | Replaced all [NSNumber numberWithBool:YES/NO] with the @YES/@NO literals. Also replaced some TRUE/FALSE with their YES/NO counterparts.
* Mark issue for s.o. to look atMax2014-12-131-2/+3
|
* Changed a few typedef enums to Apples new methodMax2014-12-132-32/+19
|
* Add stub for SPTableContentFilterControllerMax2014-12-053-0/+21
| | | | And with that we are back to a build that should not spam your Console upon launch. (Sorry for the future merge conflict) :)
* Fix "Invalid color System, labelColor" errorMax2014-12-041-14/+9
| | | | At some point Xcode decided to introduce a new variant of black named "labelColor" (which is 10.10 only). This commit changes the DBView xib back to controlTextColor instead.
* Fixes "Could not find image named" errorsMax2014-12-041-30/+0
| | | | | Switchting the IB files to Xcode 5 introduced some kind of image reference for some of our popup buttons, which was kept when we switched back to 4.6 format. However as this wasn't used in the 4.6 files before switchting to 5 either, I just removed that stuff.
* Merge pull request #2030 from origsun/masterAbhi Beckert2014-12-0211-51/+53
|\ | | | | Remove a private API in PSMTabBarCell
| * Add my name in CreditsXu Jie2014-12-011-1/+2
| |
| * Remove a private API in PSMTabBarCellXu Jie2014-12-0110-50/+51
|/ | | | | | Directly assigning to _controlView of a NSActionCell is private and not allowed by Mac App Store. An _customControlView is used to replace _controlView in PSMTabBarCell to avoid direct assigning to _controlView. This should be the last private API to remove before going to Mac App Store.
* Fix a drawing issue with the tab bar (HiDPI only)Max2014-11-301-5/+5
|
* Include tab color when duplicating tab.Max2014-11-291-0/+4
| | | | Fixes #2029
* Try to fix the legacy pagination popoverMax2014-11-291-17/+31
| | | | | | (Actually I only enabled CALayer for the container.) Also deleted a NSView that was thinking it was a NSButton, invisible, and not connected to anything. Hopefully no one will miss that thing.
* Change some split view width calculations.Max2014-11-293-7/+35
| | | | | | * Attempt to fix the "<SPSplitView: ...>: the delegate <SPSplitView: ...> was sent -splitView:resizeSubviewsWithOldSize: and left the subview frames in an inconsistent state:" message that has been plaguing my console for some time. * This commit also adds a minimum width on the main area to prevent some strange UI bugs.
* Change pagination popover to native typeMax2014-11-283-557/+3212
| | | | | | This commit switches our pagination popover to use native NSPopover on 10.7+ making it look more polished and modern. On 10.6 the old style will be used, *however* that one might still suffer from a bug caused by undefined overlapping view drawing behaviour.
* Add a "Open Table in New Window" menu actionMax2014-11-233-4165/+372
| | | | This is implemented as an alternate menu item, ie. it will appear when holding down the Option (alt) key instead of the normal "Open Table in new Tab" item. Fixes #2015
* Another attempt to fix the cog menus on 10.10Max2014-11-219-603/+8511
| | | | | | | Previously our cog menus have been Round Rect (borderless) NSPopupButtons with text aligned to the left of the image as that combination would allow us to make the box of the button exactly as big as the image. However in 10.10 Apple changed the layout so that a padding is always added on the right border of a popup button (except if the menu is attached to the left). From playing around I found the only setting to be compatible between 10.9 and 10.10 to be the button type Bevel (borderless). This commit changes all cog buttons to Bevel type in hopes those will also look the same on 10.8, 10.7, 10.6.
* Fix Tab bar colors in 10.10Max2014-11-212-9/+62
|
* Attempt to fix #1961Max2014-11-114-13/+38
| | | | | | | | | | | | | Namely this commit changes two things: 1) In the past the user manager window technically was closed after it was released (SPUserManager.m:491): [NSApp endSheet:[self window] returnCode:0]; //-> calls delegate, which calls release [[self window] orderOut:self]; This call order has now been swapped. 2) Because the delegate is invoked directly by NSApp, the release was called before other UI elements had finished their cleanup from orderOut:. The delegate callback is now put on the runloop to give other stuff priority. Requesting QA on this commit.
* Merge branch 'master' of https://github.com/sequelpro/sequelproMax2014-10-265-4107/+1150
|\ | | | | | | | | Conflicts: Source/SPGotoDatabaseController.m
| * Set Xcode version compatibility to 4.6 in GotoDatabaseDialog.xib for 10.7 buildsMax2014-10-261-117/+681
| |
| * Fix error building on 10.8.Stuart Connolly2014-10-262-4/+6
| |
| * Fix error building on 10.8.Stuart Connolly2014-10-262-27/+65
| |
| * Fix button alignment and warning.Stuart Connolly2014-10-262-3957/+398
| |
* | Tweak "Go to Database"Max2014-10-162-6/+22
|/ | | | | * Possibility to double-click an item to go there * Search is case-insensitive
* Add a "Go to Database" dialogMax2014-10-127-59/+504
| | | | | | | | The dialog enables * searching for a database by name (substring matching), * using C&P to select databases * navigating to databases not in the database dropdown * faster keyboard-based navigation
* Enable background color for content filter managerMax2014-09-181-21/+26
|
* Merge patch to disable smart quotes in editorMax2014-09-183-1/+8
|\
| * Disable Smart quotes in Bundle EditorMax2014-09-181-0/+4
| |
| * Merge branch 'disable_smartquotes' of https://github.com/uxp/sequelpro into ↵Max2014-09-182-1/+4
|/| | | | | | | uxp-disable_smartquotes
| * Fix spelling errors on method commentHoward P. Logsdon2014-09-172-3/+3
| |
| * Disable "Smart Quotes" in the query editor, among others.Howard P. Logsdon2014-09-171-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It might be useful to keep this setting on globally, but smart quotes break SQL, resulting in the requirement of having to do an undo-type-undo dance to force the editor to not replace the single (') and double (") quote characters with the ‘, ’ and “, ” characters, respectively. Since we can presume that no one will want to use these non-SQL quotation marks within SQL unless they explicitly decide to do so (in matching a varchar field, for instance), they should be off by default. Additionally, the double-dash (--) is used in SQL for inline comments, which gets substituted for the em-dash (—). Disabling this seems to be a good idea on the same argument. More info about text substitution: http://support.apple.com/kb/PH14294