kblincoe / VisualGit_SE701_2019_1

1 stars 0 forks source link

#129 - Fix the cloning UI #157

Closed encryptededdy closed 5 years ago

encryptededdy commented 5 years ago

This closes #129, closes #1, and closes #155

The UI has been updated to be unambigious and more aesthetic.

Old UI image

New UI image

encryptededdy commented 5 years ago

Thanks for the feedback, these changes have been made ^_^. Please review again as you see fit.

image

nathan-cairns commented 5 years ago

Awesome looking great :100: Just do those two small changes I pointed out in the add.repository.component.ts component and I'm happy to merge :)

encryptededdy commented 5 years ago

Unfortunately I can't see the aforementioned changes. Are they in line comments? Because I can't see any.

nathan-cairns commented 5 years ago

@encryptededdy You should be able to see them under file changes.

nathan-cairns commented 5 years ago

Hahaha nvm turns out I forgot to click submit review. You should be able to see them now!

encryptededdy commented 5 years ago

All resolved

robbierew commented 5 years ago

This is not a review but I would like these changes added soon as the code on this page effects #33

nathan-cairns commented 5 years ago

@robbierew there yah go lad!