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

Feature: Add EE for topics panel #861

Merged
merged 45 commits into from
Nov 20, 2024
Merged

Feature: Add EE for topics panel #861

merged 45 commits into from
Nov 20, 2024

Conversation

mayan-000
Copy link
Collaborator

@mayan-000 mayan-000 commented Oct 30, 2024

Description

This PR adds EE animation to the topics panel, under the Explorable Explanation tab.

Relevant Technical Choices

Testing Instructions

  • Open the PSAT panel and navigate to topics.
  • Select the Explorable Explanation tab and the animation should start working.
  • Play/pause, reset button, and speed slider should manipulate animation.
  • As the animation progresses the table should get updated with data.
  • The Epoch tabs will open a week based dummy web visit data of the user.

Additional Information:

Screenshot/Screencast

Screenshot 2024-11-07 at 14 40 15

Checklist

  • I have thoroughly tested this code to the best of my abilities.
  • I have reviewed the code myself before requesting a review.
  • This code is covered by unit tests to verify that it works as intended.
  • The QA of this PR is done by a member of the QA team (to be checked by QA).

Fixes #856

@mayan-000 mayan-000 changed the base branch from develop to feat/ee-base October 30, 2024 09:38
@mayan-000 mayan-000 marked this pull request as ready for review November 7, 2024 09:24
@mayan-000 mayan-000 requested a review from mohdsayed November 7, 2024 09:24
Base automatically changed from feat/ee-base to develop November 7, 2024 11:30
@mohdsayed

This comment was marked as resolved.

@mohdsayed

This comment was marked as resolved.

@mohdsayed
Copy link
Collaborator

@milindmore22 Lets QA this PR

@milindmore22

This comment was marked as resolved.

@mohdsayed
Copy link
Collaborator

mohdsayed commented Nov 18, 2024

@mayan-000 Some more items to be done in a separate PR as we discussed.

  • Pause topics EE when you go other tab.
  • Make the topics clickable and take it to the relevant Taxonomy node.
  • Add arrow to connect topics to box, right now they are looking disconnected.

@mohdsayed mohdsayed added this to the v0.12 milestone Nov 20, 2024
@mohdsayed mohdsayed merged commit 392735e into develop Nov 20, 2024
6 checks passed
@mohdsayed mohdsayed deleted the feat/ee-topics branch November 20, 2024 06:49
amovar18 pushed a commit that referenced this pull request Dec 12, 2024
* Add header section with button, slider, add table tray with tabs and resizable component

* Add epoch table and more tabs

* Move components and reorganize

* Setup ee for PA

* Add PA tables to tray config

* Use half pointers for slider

* Use props for tab state

* Move active tab state a level up

* Add callbacks to retrieve vertical bar color hash

* Add query state

* Change colors for dark mode

* Add checkboxes for PA

* Change checkboxes layout

* Setup topics animation

* Break p5 into files and use with React

* Draw info box

* Update container sizes

* overflow to resize

* Handle play/pause and reset callbacks

* Handle animation speed step

* Handle user visit data change

* Render data in the tables

* UI changes

* Render border conditionally

* Update border

* Animation, table UI changes

* Refactor code

* Update UI, handle unexpected flow by updating calls

* Auto change active tab

* Extend timeline

* Make animation continuous with animation consitency if moved to previous tab

* Refactor callback and conditionally call user visit

* use ref for not rendering on animation var value change

* Dont overlap with horizontal lines

* Update image data

* Add line  from circle to infoox

* Update color circles sizes and text placement

* handle restart of animation on complete when replay is clicked

* Update callbacks for user visit
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.

3 participants