-
Notifications
You must be signed in to change notification settings - Fork 4
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
Merge new UI into production branch #130
Merged
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
Display FirefoxConnect as "Firefox for Echo Show"
* adds barebones from graph-paper story * integrates config file * adds back Logo; further tweaks to menu items; implements cache + querying * adds Loading Spinner * finesses error / loading states * adds calls to action (feedback, shortcuts, docs, file an issue) in Sidebar * adds esc * further tweaks to Shortcuts menu * styling tweaks * adds back querystring state loading * updates store to have a no-local querying approach to support transitions within Explore.svelte * adds back in TimeAxis.svelte * updates and tweaks some layout pieces * updates the mouseovers to account for bug fixes (contingent on a new graph-paper PR) * adds back in caveat system * adds smoothing option * adds hash routing for future improvements * fixes the hashbang routing with query parameteres * wires up explore body menus to read from querystring * metric selector; moves some compoinents to src/components/controls * compartmentalizes the mouse interactions * adds back in markers * cleans up modal implementatino * sidebar tweaks * adds a few more affordances around YoY + mouseovers; cleans up all menus * feature-complete * adds disabledMetrics and disabledDimensions * makes lists keyboard-accessible * restyles sidebar in anticipation of multiple views * migrates to graph-paper 0.0.0-alpha.14 * improves mouseover performance * further design tweaks * adds in changes to usage criterion from prototype branch * adds newer clickOutside action * reset start and end dates on usage criterion change * removes extraneous comments * updates a handful of dependencies * adds back tooltips into graphs * adds TitleMenu.svelte component * Consolidates routing / config information * adds disabled functionality * adds rudimeentary scrollIntoView support * adds CHANGELOG.md
* moves around a few components; fixes DatePicker animation issue * collapses DatePicker on reset * removes buttons in explore when focused on a single metric
* Switch to a Node-based eslint-health script * Run ESLint against Svelte files
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
Resolves #112 Relies on mozilla/bigquery-etl#1352 to make JP available in the underlying tables.
…use the route is not codified in the hash. (#124)
This same documentation exists in the UI, so I think it's best to remove it here.
Merge new UI for production
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.
After this is merged, we'll announce in the
#gud
channel about the new URL and the UI changes. We will also highlight the caveat that some URL params have changed, so links may need to be a bit different for the new UI.