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

72 make legend box component #82

Merged
merged 25 commits into from
Jul 3, 2024
Merged

72 make legend box component #82

merged 25 commits into from
Jul 3, 2024

Conversation

amaliejvik
Copy link
Contributor

@amaliejvik amaliejvik commented Jul 3, 2024 •
edited
Loading

  • A sticky box located in the upper left corner that displays an overview of which components are present in the scene in Latex
  • Can be collapsed into a small square, and then extended again by clicking the arrow icons
  • When hovering over elements of type Plot, Point and Circle in the LegendBox, the corresponding Components in the scene are highlighted
  • Support for State variables, which LaTeX-strings can rely on. Further, LaTeX-strings in LegendBox will update according to the state variables

The following information is currently configured to be displayed for the given element types:

  • Plots: <Plot name> : <Plot expression>
  • Points: <Point name> : (x,y)
  • Stat variables: <State name> : <State value>
  • String: <String with state variables> : <String where state variables are replaced with actual current state values>
  • Remaining components: <Component ClassName>
Skjermbilde 2024-07-03 kl 15 37 06

Closes #72

amaliejvik added 20 commits June 27, 2024 15:55
@amaliejvik
Beginning of LegendBox
af7191d 
- Plots get names starting from f and following the alphabet
- Plots get colors from a default list that cycles
- Some styling

Related to #72
@amaliejvik
Add expand-collapse function and HideFromLegendBox function
3c0d548 
Related to #72
@amaliejvik
Move Plot-funcName logic out of Core.ts and into Plot.ts
565a48b
@amaliejvik
Add possibility for having update() function on GuiComponents as well
4af059a
@amaliejvik
LegendBox updates if a plot is dynamic (moves)
f277f9d 
Related to #72
@amaliejvik
Make expand-collide button work together with dynamic function names
d3c9526
@amaliejvik
Make LegendBox size adjust based upon how many plots it holds
02d6ac1 
Related to #72
@amaliejvik
Generalize legend box
724f10f 
- Add getName() function on Component class, which returns its ClassName if not defined to something else in the component itself and an empty string if component.name is not defined either
- Components can have a displayText() function, which defines which text should be present in LegendBox
- In the LegendBox, each row consists of <dot of plot color | black dot> + <component.name>. If displayText exists on component, this is displayed after <component.name> (ex. function expression for plots and coordinates for point)
- Add emitter for Point, such that LegendBox updates when point is dragged around

Related to #72
@amaliejvik
Add name to all Component classes
aae68e0 
Related to #72
@amaliejvik
Add name to Fraction
1552950 
Related to #72
@amaliejvik
Add hover-functionality for LegendBox and Components
1669de5 
Related to #72
@amaliejvik
Add support for different types of icons in LegendBox and a color-fix
24d69d1 
Related to #72
@amaliejvik
Just some testing
2095d78
@amaliejvik
Another color fix
f23a2dc
@amaliejvik
Add functionality for having state variables and latex-text dependent…
2ae652f 
… on these variables in the legendBox

(A draft that works, code must still be refined)

Related to #72
@amaliejvik
Generalize State.ts
5be5ab5 
Related to #72
@amaliejvik
Hover on Points bux-fix
afb4323
@amaliejvik
Refactoring/clean-up of code
7a24680 
Related to #72
@amaliejvik
Further generalization of State.ts and a Style fix
0edf6cf
@amaliejvik
Final clean-up
3b04f88
@amaliejvik amaliejvik self-assigned this Jul 3, 2024
@amaliejvik amaliejvik linked an issue Jul 3, 2024 that may be closed by this pull request
Make legend box component #72
Closed
@amaliejvik amaliejvik requested a review from jonasdeluna July 3, 2024 13:38
jonasdeluna
jonasdeluna approved these changes Jul 3, 2024
View reviewed changes
Copy link
Member

@jonasdeluna jonasdeluna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, just fix the lint/prettier 🚀

@amaliejvik amaliejvik merged commit 9fa6c62 into master Jul 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonasdeluna jonasdeluna jonasdeluna approved these changes

Assignees

@amaliejvik amaliejvik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make legend box component
2 participants
@amaliejvik @jonasdeluna