Skip to content
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

Add share option, refactoring + minor tweaks #15

Merged
merged 14 commits into from Apr 27, 2018
Merged

Conversation

Aecasorg
Copy link
Collaborator

@Aecasorg Aecasorg commented Apr 26, 2018

Added share option for Pastebin links.
Moved post functions to PastebinHelper class.
Smoothed transitions between all views to give more uniform flow.

…ns menu syntax selector and the paste view popup syntax selector.

- Fixed so that the title on the paste view popup syntax selector also updates when a syntax is chosen using the textfield search.
Shortened the post method slightly with a couple of ternary operators.
Reversed Previous Pastes view order with latest paste at top now.
…on border attributes in the Attributes Inspector

- Added borders to buttons on main view so buttons are more clearly... buttons :P
- Separated buttons further from each other to stop possible miss-clicks
- Edited text on buttons to make it more... "button like"? :P
…nks. NOTE: This needs to be tested on an actual device (I can't) as many of the apps that can be shared to are missing on simulator. Share should ideally be able to open in directly Safari however also as a string link into messenger, email and also external apps such as Twitter, Facebook, Slack among others. This feature needs to be tinkered with.
…of thin border.

Set all segues to have same smooth transition to make it more uniform.
…fit in better with rest of app.

Changed transition to TextSelectionViewController to cross fade in line with rest of app.
Stacked main view buttons so that they, together, center on the screen rather than top button center.
@Aecasorg Aecasorg changed the title Mainly refactoring + minor tweaks Add share option, refactoring + minor tweaks Apr 26, 2018
@jonluca jonluca merged commit 6f06ca1 into jonluca:master Apr 27, 2018
@Aecasorg
Copy link
Collaborator Author

Aecasorg commented Apr 27, 2018 via email

@jonluca jonluca mentioned this pull request Apr 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants