Allow users to toggle modifiers #38
Merged
Conversation
This file contains 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
damonhook
added a commit
that referenced
this pull request
Feb 21, 2020
…ce improvements (#35) ## Additions - Users can now toggle profile and target modifiers using a toggle switch (fixes #38). ## UI - Remove Beta Tag from non beta features - Small UI overhaul for Desktop (Mostly related to the drawer) - Add Warcry Statshammer links and buttons (fixes: #23) - Change SVG Text Object -> Path (fixes #25) - Use SVGIcon instead of loading through img src - Make the Left navigation bar responsive based on the following criteria (fixes #34) - **Large:** Show the full drawer - **Medium**: Show a Rail but let users open the drawer - **Small:** Hide the rail, let users open the drawer - Add an Average damage graph to the `Metrics` tab in the `Simulations` screen for easier reference ## API - Use a calculated population max (The actual max) rather than basing it on the max value of the simulated sample (fixes: #30) - This has had the effect of vastly improving the accuracy of the simulated metrics. - Also use the population mean, rather than the sample mean - Remove Median as the way the data is distributed, it is usually extremely close to mean anyway - Calculate frequency table on the fly, rather than creating a large results array first (fixes #32) - Completely refactor the response structure of the `/api/simulate` and `/api/simulate/save` endpoints for better visibility and extensibility. - Added Sentry to API to capture errors (Fixes: #39) ## Notes - This UI overhaul was more focused on navigation, the next one will be more about layout - Though it may not look it, this release had a huge amount of behind the scenes changes - Since the 2.0.0 branch, I have been using Github projects to greater effect, this means you can watch the project for a specific version to see the currently planned items, and their progress - [v2.0.0](https://github.com/damonhook/aos-statshammer/projects/1) This one - [v2.1.0](https://github.com/damonhook/aos-statshammer/projects/2) The next one - [v2.2.0](https://github.com/damonhook/aos-statshammer/projects/3) This one
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.
Additions