Commit Graph

84 Commits

Author SHA1 Message Date
BrianLitwin
fe5ec6a00f updates/tests detectShortlink method (#1971)
* updates/tests detectShortlink

* Updates Int character check

* deleted redundant Unit Tests
2018-08-05 12:24:37 -04:00
Ryan Nystrom
e56c29a9a6 Redesigned Issues interface (#1983)
* clean up title model

* update title style

* organize issue SC method

* remove background colors from styled text in issues

* collection view bg white

* remove card inset and clean up snap superview code

* clean up label design

* thread is root and change base font size

* remove borders on comment cells

* adjust collapse design

* adjust comment line spacing and make non-root body smaller

* refactor detail view

* viewer header background color

* more buttons in reaction cell

* vertical spacers working

* design basically finished;

* scroll to bottom accounts for inset

* PR review background color white

* fixup PR review comments

* spacing on merge and review

* readme background white

* rename to just "spacer"

* horizontal spacing

* fix tests
2018-07-27 11:03:09 -04:00
BrianLitwin
08221f9191 Unit Test IssueCommentTable dimensions (#1864)
* Unit Tests IssueCommentTable dimensions

* updates markdown table tests
2018-07-09 14:52:41 -04:00
Ryan Nystrom
1ac9348852 update to new StyledTextKit lib name (#1849) 2018-06-02 23:14:11 -04:00
Ryan Nystrom
4da4ff4a92 Disable flaky tests (#1838) 2018-05-29 08:21:17 -04:00
BrianLitwin
650756fe1d updates add reviewers url path (#1832) 2018-05-25 19:50:28 -04:00
Ryan Nystrom
e9899fb935 Delete NSAttributedStringSizing (#1826)
* Delete NSAttributedStringSizing

* compile
2018-05-21 11:15:38 -04:00
Ryan Nystrom
1beccb9508 Simply merge status logic (#1818) 2018-05-19 19:07:42 -04:00
BrianLitwin
3d3693efad Issue resolved with url description containing shortlink (#1813) 2018-05-16 07:23:40 +02:00
Ryan Nystrom
a4f98163e5 Merge pull request #1802 from BrianLitwin/Issue#120Fixed
Brackets inside link text fixed
2018-05-11 09:09:22 -04:00
Ryan Nystrom
de801c5601 Merge pull request #1780 from GitHawkApp/vulnerability-support
Add support for vulnerability alerts
2018-05-10 09:09:25 -04:00
Bas Broek
b61324baa7 Update tests 2018-05-10 10:31:10 +02:00
Brian Litwin
f3c65d0f2c Brackets inside link text fixed 2018-05-09 23:32:00 -04:00
BrianLitwin
27b83b666d update bookmark nav item state between viewDidLoad and Issue Load (#1791) 2018-05-07 20:26:59 -07:00
Ryan Nystrom
b02c7cf0ad functioning watch app 2018-04-29 21:38:25 -04:00
Ryan Nystrom
17863c258e fix tests (#1755) 2018-04-22 16:57:54 -04:00
Ryan Nystrom
178d7700b7 Remove MMMarkdown dependency (#1729)
* replace old markdown callsites

* unit tests working

* remove unused parsing

* final removal of old parsing, build and tests succeed

* remove dependency

* capture readme ref branch
2018-04-14 23:12:03 -04:00
Ryan Nystrom
5fd68c1e6a Rewrite markdown parsing with cmark (#1704)
* add cmark lib, build succeeds

* building and running (with dupe asserts)

* list newlines, quote styles

* tables working in cmark

* fix table bg color math and code bg color

* code block highlighting only when theres a language

* share code block logic

* mentions need @

* update cmark lib

* shortlinks

* detect issue shortlinks

* simplify and correct shortlinks

* remove html comments

* tests passing
2018-04-07 23:27:14 -04:00
Ryan Nystrom
62e2763a85 Refactor markdown parsing to use text lib (#1674)
* milestones refactored with styled text

* reference refactored

* refactor commit references

* fix styled text bug

* request event refactored

* refactor titles

* warm caches for bg-gen models

* delete old text cell

* move shortlinks into MMMarkdown

* almost there

* build green

* running and trimming whitespaces

* styled text working

* fix tests

* tests passing
2018-03-31 10:12:51 -04:00
Ryan Nystrom
83acbea441 Start refactoring text views with StyledText (#1658)
* refactoring with styledtext

* issue rename refactored

* fix tests
2018-03-17 18:55:08 -04:00
Bas Broek
755a9fe35d Move references to old url (#1652) 2018-03-17 11:41:57 -04:00
Bas Broek
ddddb6363f Add merge status tests (#1604)
* Add merge status tests

* Introduce containsAll and containsNone

* Rename containsAll to containsOnly
2018-03-04 21:36:25 -05:00
Bas Broek
9bd61354dd Remove HangingChadItem (#1605)
Referencing https://github.com/rnystrom/GitHawk/pull/1327#discussion_r172028966
2018-03-03 17:03:11 -05:00
Ryan Nystrom
803f9e29c5 Revert "Change product name to GitHawk (#1545)" (#1562)
This reverts commit 42caf56acd.
2018-02-21 10:20:32 -05:00
Ryan Nystrom
42caf56acd Change product name to GitHawk (#1545)
* Change product name to GitHawk

* rename module
2018-02-21 08:23:32 -05:00
Ryan Nystrom
0001920b75 Refactor text styles to use StyledText (#1490) 2018-02-04 19:09:42 -05:00
Bas Broek
671ffe8d72 Distinguish owner from repository in dropdown actionsheet (#1411)
* Distinguish between owner and repo in action sheet

* Bonus: fix test warning
2018-01-14 12:11:44 -05:00
Ryan Nystrom
56864889a7 Manage button scrolls to bottom to reveal actions (#1327)
* new prototype of managing UX

* add tested helper for calculating item width without hanging chad

* adjust math to even last row when theres a chad
2018-01-02 07:15:48 -05:00
Ryan Nystrom
e0c8d5c37e Parse issue shorthand within MMMarkdown (#1270)
* parse github shorthand links within MMMarkdown

* all sorts of shorthand links
2017-12-17 09:18:05 -05:00
Ryan Nystrom
270903e1ae refactor dir and blob paths, add tests (#1165) 2017-12-03 16:36:43 -05:00
Ryan Nystrom
178b37eb27 Render checkmarks (#1078)
* add assets

* put marker for checked and unchecked

* attachment working

* export smaller assets

* use lighter blue for checked

* thread can-update, undo accident

* nudge down checkmark attachment

* add tests
2017-11-26 12:27:32 -05:00
James Sherlock
0dffae020b API Keys read from environment variables (#1075)
* Update client keys to use environment vars

* Extra logging to assist with debugging

* Add other files to test target (temporary solution)

* Update Project Target References

* Add arguments and share scheme

* Disable failing FBSnapshot tests

* If at first you fail, wait 20 minutes for a build and then try again

* Update test to also ensure it's not using the template keys

* Update gitignore

* Remove intentional failing test

* revert pod changes

* revert podfile
2017-11-24 20:51:31 -05:00
Ostap
d6cba1e280 Added FBSnapshotTestCase integration. Added initial tests for AttributedStringView (#688) 2017-11-22 09:24:57 -05:00
Ryan Nystrom
dac2c00c62 Bookmark cell refactor for consistent UI (#980)
* Bookmark cell refactor for consistent UI

* remove useless test
2017-11-18 16:47:12 -05:00
Hesham Salman
b18858ac6e Bookmark subtitle color (#957)
Updated the bookmark subtitle color and the issue # color to medium-gray
2017-11-16 08:46:25 -05:00
Hesham Salman
7b25f6acb0 Picked up the easy bits of Ryan's review #865 (#877)
* Picked up the easy bits of Ryan's review #855

- Replaced "// MARK: -" with "// MARK:"
- Added final to classes
- Added newlines for readability
- Computed and stored diffIdentifier of BookmarkViewModel

* Removed NSObject from BookmarkViewModel

* Moved icon logic to the configure block of the BookmarkCell
2017-11-07 08:45:31 -05:00
Hesham Salman
1e2e0c1698 Chore/bookmark rewrite (#865)
* Unified store behavior, new bookmark model

- Added a `Store` protocol to the project with default implementations
  that should be sufficient for most models. (#855)
- Switched images in the notification type category to #imageLiterals,
  so we can avoid the optionality of the image.

* Implemented BookMark ViewModel

* Created Bookmark Collection Cell

* Created Bookmark View Controller

Done:
  - Displays empty state
  - Displays bookmarks
  - Allows for adding of new bookmarks
  - Navigating to bookmarks

In flight:
  - Clear All

TODO:
  - Search
  - Cleanup old files & remove SwipeCell hacks

* Unified search and clear all behavior

The search bar and clear all buttons between the search and bookmarks
page had different styles and behavior. This PR:
  - Makes a generic clear-all header
  - Gives one to each VC
  - Makes them behave the same to the user re: visibility etc

* Re-implemented filter

* Deleted old files

* Attributed String Sizing

* Removed bookmark store tests

* Reloading bookmarks & namespacing

Can now reload bookmarks through the listener pattern (#773, #783).
Bookmark store is now correctly namespaced again.
2017-11-06 09:03:29 -05:00
Ryan Nystrom
eb0e887f09 Fix bookmarks tests (#871) 2017-11-05 17:58:43 -05:00
Hesham Salman
814f6d6c1c Fixed spec (#857) 2017-11-05 12:39:45 -05:00
Ryan Nystrom
0158f3f582 Empty view for bookmarks (#851)
* Empty state background for Bookmarks

* psd changes
2017-11-04 21:47:41 -04:00
James Sherlock
63cc844b0a Use repository default branch rather than master for file view (#849) 2017-11-04 21:42:54 -04:00
Rizwan Mohamed Ibrahim
66481af678 Sorted/Ordered Bookmarks (#825)
* [ADD] Sort bookmarks before converting into Array

* Changes from Set<BookmarkModel> to NSMutableOrderedSet
2017-11-04 10:54:34 -04:00
Rizwan Mohamed Ibrahim
684d643d46 Update bookmarks without reloading view (#783)
* [FIX] Update bookmarks without reloading view

* [ADD] Listener wrapper and fix speeling mistake

* [ADD] BookmarksStoreListener tests
2017-10-31 19:59:37 -07:00
Rizwan Mohamed Ibrahim
199ce48047 [ADD] Bookmarks per user account (#738) 2017-10-27 10:55:22 -04:00
Rizwan Mohamed Ibrahim
fba236450b Adds options to remove bookmark (#696)
* [ADD] options to remove bookmark

* [ADD] Move bookmark actions to bottom of action sheet
2017-10-24 21:43:22 -04:00
Hesham Salman
2ec7ba2753 Preventing the search race condition instead of responding to it (#691)
* More explicit approach to search triggers

Cutt reliance off of the singleton store

* Cleaned up optional code-smell

Comments
2017-10-23 17:08:55 -04:00
Hesham Salman
f213bac924 Thin SwiftLint ruleset (#704)
* Thin SwiftLint ruleset

* Disabled trailing_whitespace rule
2017-10-23 16:58:09 -04:00
Rizwan Mohamed Ibrahim
bd10b4e032 Add Bookmarks tab (#601)
* [ADD] Bookmarks for repo, issues and pull request

* [FIX] Bookmark model decoding from archive

* [ADD] Clear all option for bookmarks

* [ADD] Search bookmarks with search bar

* [ADD] Swipe to delete for bookmarks (only when search bar is not active)

* [ADD] Bookmark cell accessible and some cleanup

* [ADD] Incorporate review comments

* [ADD] Reuse notification type instead of bookmark type

* [ADD] Incorporate review comments
- Enum over struct on bookmark model
- Github bookmark tab icon
- Smooth deselection, Single tap to scroll to top, cleanup
-
2017-10-22 14:00:50 -04:00
James Sherlock
bde290f192 Image Upload for Bug Reports (#475)
* WIP: Basic image upload for bug reports

* Added image upload UI, Improved flow, Sped up process, Accurate information is passed through

* Improved navigation structure and transitions

* Tidy up of code, enable actual submission to Imgur

* Update to use table view

* Minor changes as per @BasThomas code review

* Remove all classes from Test target, Added @testable

Should fix Buddybuild issues

* Tweaks as per Ryan code review, Moved image upload logic to shared controller, added flag to toggle upload action
2017-10-22 13:58:03 -04:00
Hesham Salman
866418ec8a New tests, fixed failing tests (#645) 2017-10-22 11:58:07 -04:00