-
-
Notifications
You must be signed in to change notification settings - Fork 112
Clone repository: Show progress #1608
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
Closed
+159
−26
Closed
Changes from all commits
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
e1d2e5c
Add clone button to sidebar
jeremypw ebb8ed1
Sketch out functionality
jeremypw 5200d9d
Fix action, Tweak regex
jeremypw 01d124f
Various corrections
jeremypw e39eac8
Clone repository
jeremypw 7e3da04
Reformat, local repo name, make active option
jeremypw 3b4ec49
Merge branch 'master' into jeremypw/implement-clone
jeremypw 328c75b
Merge branch 'master' into jeremypw/implement-clone
jeremypw a267a98
Fix lint
jeremypw 7881aee
Merge branch 'master' into jeremypw/implement-clone
jeremypw 85b2bb3
Merge branch 'master' into jeremypw/implement-clone
jeremypw 2251850
Allow narrower sidebar
jeremypw 15c6fc5
Merge branch 'master' into jeremypw/implement-clone
jeremypw ea1a356
Update src/Dialogs/CloneRepositoryDialog.vala
jeremypw 576ca43
Update src/Dialogs/CloneRepositoryDialog.vala
jeremypw 9c38d00
Replace local folder path entry with button linked to filechooser
jeremypw 5287f59
Set mnemonic widget for CloneEntry label
jeremypw 9f8bf46
Fix construction as advised, add comments
jeremypw 497bac7
Better variable names
jeremypw b4c79ed
Remove activates_default = false;
jeremypw 55490d2
Rewording
jeremypw 7a4bf5d
Use "emblem-downloads" icon
jeremypw a59c64b
No space before ellipsis
jeremypw 6138cfa
Do not use Gtk.Button.from_icon_name
jeremypw 0582c77
More explicit labels
jeremypw 50293b5
Merge branch 'master' into jeremypw/implement-clone
danirabbit ec4b2d0
Simplify
jeremypw 4dd6f6d
Merge branch 'master' into jeremypw/implement-clone
jeremypw ffbb883
Show all only on content_box
jeremypw 636d871
Use box padding not margin
jeremypw 7cadd28
CloneRepositoryDialog: design tweaks (#1584)
danirabbit 2c315a2
Move buttons from actionbar to projectchooser (#1583)
danirabbit 1bfa1aa
Check URL is in correct form using function not regex
jeremypw d428416
Merge branch 'master' into jeremypw/implement-clone
jeremypw 3273fe5
Prefill local name; improve NAME_REGEX
jeremypw 67be8d9
Restrict project folder names
jeremypw 273113c
Merge branch 'master' into jeremypw/implement-clone
jeremypw 731a05a
Fix extraneous comment
jeremypw e3a22b9
Remove set as active checkbox for now
jeremypw 25bbd0e
Use SYNC_CREATE
jeremypw 4d66c36
Fix unneeded try-catch
jeremypw 379ef30
Move cloning into response callback
jeremypw c2fe76f
Close dialog before cloning starts
jeremypw 22e57b7
Add TODO comment
jeremypw 7d60208
Fix undefined variable
jeremypw 2595107
Handle clone failure with dialog - option to retry
jeremypw 1454a5b
Handle no current active project
jeremypw 1b3232d
Persist projects folder and remote entries
jeremypw 0ae3fb9
Fix lint
jeremypw e949241
Merge branch 'master' into jeremypw/implement-clone
jeremypw 5536cc4
Merge branch 'master' into jeremypw/implement-clone
leonardo-lemos 784460d
Merge branch 'master' into jeremypw/implement-clone
zeebok 6915fce
Merge branch 'master' into jeremypw/implement-clone
jeremypw 01346c9
Add on accelerator to open a project
jeremypw 4119847
Merge branch 'master' into jeremypw/implement-clone
danirabbit db0e6f5
Merge branch 'master' into jeremypw/implement-clone
jeremypw af36e42
Set parent to error dialog, fix spelling
jeremypw e463c9f
Make clone function async
jeremypw 9d95150
Include success messagedialog
jeremypw d049990
Only construct parameters in Object ()
jeremypw 1a09d91
Use a thread for cloning, show spinner
jeremypw fc536e0
Remove unneeded throws Error
jeremypw 49e95a9
Fix some coding issues
jeremypw e4cf12b
Fix typo
jeremypw 4469d6c
Show spinner on separate page; hide buttons
jeremypw 1afe983
Cleanup comments
jeremypw 6779e66
Reorganise, reinstate default action
jeremypw ebcfd41
Implement some remote callbacks
jeremypw ea582db
Merge branch 'master' into implement-clone/remote-callbacks
jeremypw 4e35200
Remove extraneous
jeremypw 17bb019
Merge branch 'master' into implement-clone/remote-callbacks
jeremypw 6f00e24
Merge branch 'master' into implement-clone/remote-callbacks
jeremypw d5ce749
Show cloning spinner in ChooseProjectButton
jeremypw 1e6872f
Add cloning success toast to sidebar
jeremypw 1e31a8d
Merge branch 'master' into implement-clone/remote-callbacks
jeremypw 558bb1a
Merge branch 'master' into implement-clone/remote-callbacks
zeebok 9f7af0f
Merge branch 'master' into implement-clone/remote-callbacks
jeremypw File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This class is getting way too large in my opinion. If these widgets are solely to indicate clone progress, they should be marked as so.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think less than 400 lines is particularly large but I'll see whether anything can reasonably be split out. Really need to agree the UI for showing progress before worrying to much about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not about the lines. It's about the responsibilities.
1 class should have only 1 responsibility.
It's said that our brains can only keep track of 10 things at once. If it's more, it can't handle it. Of course, this depends on the case.