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.

Sign up for GitHub

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

Jump to bottom

WIP: Playground redesign #144

Merged
merged 56 commits into from
Jul 15, 2024
Merged

WIP: Playground redesign #144

merged 56 commits into from
Jul 15, 2024

Conversation

JuroUhlar
Copy link
Collaborator

@JuroUhlar JuroUhlar commented May 22, 2024 •
edited
Loading

JIRA: https://fingerprintjs.atlassian.net/browse/INTER-510
Demo: https://staging.fingerprinthub.com/playground

  • Redesigned the Playground according to the original Figma design unifying the esthetic with the main website
  • Added How to use this demo section featuring a new Accordion component
  • Added a Learn more section and made the tiles switch to horizontal scrolling behavior on mobile devices
  • Signal tables are collapsible now using a new Collapsible component
  • Improved latency perception by showing the visitor ID right away while waiting for the Server API result
  • Improved JSON viewing experience:
    • Researched and found a JSON viewer library with (hopefully) sufficient customization options
    • Copy button for the entire JSON and individual objects within
    • JSON nodes can collapse and expand on click
    • Better (more precise) syntax highlighting
    • The new library required simplifying some of the e2e tests but I think they still work well enough
    • Unified scrollbar appearance across browsers using a new ScrollArea component
  • Moved Playground to the app router
  • Smarter use of accuracyRadius - if large, we only show the user their country, and the map is zoomed out
  • Refactored FancyNumberedList into a reusable component
  • Refactored ResourceLinks into a reusable component
  • Fixed React mark-up errors in SVG components
  • CI: Fixed E2E report-status job condition

JuroUhlar added 30 commits May 22, 2024 19:37
@JuroUhlar
chore: move playground to app router
4dab39b
@JuroUhlar
feat: add hero section
de97c63
@JuroUhlar
feat: visitorId box
c859699
@JuroUhlar
chore: move components into playground
41e814d
@JuroUhlar
Merge branch 'main' into feat/playground-redesign
8adc44b
@JuroUhlar
feat: style table v1
be4ccf8
@JuroUhlar
feat: table link, prevent icon wrapping to a separate line
d042049
@JuroUhlar
Merge branch 'main' into feat/playground-redesign
923951b
@JuroUhlar
feat: fix external link icon color and location text
242afb3
@JuroUhlar
feat: nicer map
2d4686e
@JuroUhlar
feat: fix map height, suspect score n/a
2cda22d
@JuroUhlar
feat: analyze again button
ace75b8
@JuroUhlar
feat: accordion wip
076505f
@JuroUhlar
feat: improve accordion
c819b6e
@JuroUhlar
feat: finish how to use this playground
dd29ee3
@JuroUhlar
chore: update Next
d767f71
@JuroUhlar
feat: finish how to use this playground
7613f13
@JuroUhlar
feat: json section
76d15f0
@JuroUhlar
ci: fix report status job
d5125da
@JuroUhlar
feat: layout fix
b01180c
@JuroUhlar
feat: Learn more section
9f69b22
@JuroUhlar
feat: scrolly scroll
4c6f33d
@JuroUhlar
feat: improve scrollbar position
98589ec
@JuroUhlar
feat: finally fix scroll area
a35d4f7
@JuroUhlar
feat: basic copy button
e52d275
@JuroUhlar
feat: copied state for copy button
e87dfe4
@JuroUhlar
styles: running intelligence padding
28fa95f
@JuroUhlar
styles: responsiveness whitespace
715ebf9
@JuroUhlar
styles: responsiveness font-size
ff55ef6
@JuroUhlar
styles: more responsiveness
80d9251
@JuroUhlar JuroUhlar marked this pull request as ready for review July 2, 2024 13:24
@JuroUhlar JuroUhlar requested review from ilfa and TheUnderScorer and removed request for ilfa July 2, 2024 14:20
ilfa
ilfa previously approved these changes Jul 3, 2024
View reviewed changes
src/client/components/common/componentUtils.tsx Outdated Show resolved Hide resolved
TheUnderScorer
TheUnderScorer reviewed Jul 4, 2024
View reviewed changes
e2e/playground.spec.ts Outdated Show resolved Hide resolved
e2e/playground.spec.ts Outdated Show resolved Hide resolved
src/client/components/common/CodeSnippet/CodeSnippet.tsx Outdated Show resolved Hide resolved
src/client/components/common/componentUtils.tsx Outdated Show resolved Hide resolved
TheUnderScorer
TheUnderScorer previously approved these changes Jul 8, 2024
View reviewed changes
ilfa
ilfa previously approved these changes Jul 8, 2024
View reviewed changes
@JuroUhlar
Feat/final redesign touches (#147)
5049a57 
* feat: external link animation

* feat: custom spinner

* feat: running intelligence animation

* fix: move refresh button to the top

* feat: add remote control smart signal

* feat: add velocity smart signal

* left align hero section on mobile

* fix accordion color

* fix build

* feat: use the same mapbox tiles as the main website

* ci: run check even when PR is not into main

* fix: spinner size
@JuroUhlar JuroUhlar dismissed stale reviews from ilfa and TheUnderScorer via 5049a57 July 12, 2024 12:31
@JuroUhlar
Playground Layout: Put visitor ID in Identification column (#148)
f96d384 
* feat: external link animation

* feat: custom spinner

* feat: running intelligence animation

* fix: move refresh button to the top

* feat: add remote control smart signal

* feat: add velocity smart signal

* left align hero section on mobile

* fix accordion color

* fix build

* feat: use the same mapbox tiles as the main website

* ci: run check even when PR is not into main

* feat: put visitor ID box in the identification column

* fix mobile layout

* self-review: remove comments

* test: fix locator

* fix map z index

* fix: disable scroll restoration on playground

* feat: add table animations

* fix: remove !important
@JuroUhlar JuroUhlar requested a review from ilfa July 15, 2024 11:00
@JuroUhlar JuroUhlar merged commit abea4e7 into main Jul 15, 2024
5 checks passed
@JuroUhlar JuroUhlar deleted the feat/playground-redesign branch July 15, 2024 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilfa ilfa ilfa approved these changes

@TheUnderScorer TheUnderScorer TheUnderScorer left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JuroUhlar @ilfa @TheUnderScorer