-
Notifications
You must be signed in to change notification settings - Fork 4
UI redesign pt1 #39
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
Merged
Merged
UI redesign pt1 #39
Conversation
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
- Refactor ToolsView component with new layout - Added sidebar functionality with animation
* minor ui fixes
* no sidebar on mobile, instead show a list of utility cards that are clickable to go to the utility page. If on mobile and not on the '/tools' page, only show the child component and not the parent
* Bring in and set up ace code editor for JSON input * Enable syntax highlighting on JSON input to provide valuable user feedback when JSON is not valid * Bring in vue-debounce to debounce auto-convert utilities * minor mobile UI fixes * bring in vue-toastify to provide user valuable toast messages when certain actions are performed, such as copying to clipboard
* add vertical padding to TwoPaneLayout * align color converter conversion boxes * rename LineSort variables to camelCase and add vertical padding between elements
✅ Deploy Preview for celadon-beignet-3132dc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
* fix StringInspector prompts * wrap PHP>JSON conversion in try/catch so we don't clog up the console * use vue-json-pretty types
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Starts to fix #32.
This is the initial effort for overhauling the design and user experience of the app.
At this point, we have anough to warrant a deployment. The app feels greatly improved from it's previous state.
More to come, but going forward we'll likely focus on a single page/utility at a time, and doing some sort of a continuous deployment going forward. Keeps commits small and commit often!
If you notice any bugs, undesired behaviour, or overall general feedback, please comment, open an issue, or put up a PR!