Commit Graph

5 Commits

Author SHA1 Message Date
Shahan Nedadahandeh 1a98bea483 Added min width to missing graphs (#176)
continuous-integration/drone/push Build is passing Details
Closes #129
Closes #120

Co-authored-by: shahanneda <shahan.neda@gmail.com>
Reviewed-on: #176
Reviewed-by: Mark Chiu <e26chiu@csclub.uwaterloo.ca>
2023-02-13 18:58:06 -05:00
Mark Chiu c5c84176ec Fix boxplot labels and incorrect questions order (Closes #147) (#174)
continuous-integration/drone/push Build is passing Details
* Refactored Boxplot to remove most hardcoded values, see updated version below:
* Reorder certain questions (see issue)
* Aligned boxplot labels correctly

![image](/attachments/8414d9be-bada-4ab8-813c-705505e551af)

Co-authored-by: e26chiu <e26chiu@csc.uwaterloo.ca>
Reviewed-on: #174
Reviewed-by: Shahan Nedadahandeh <snedadah@csclub.uwaterloo.ca>
Co-authored-by: Mark Chiu <e26chiu@csclub.uwaterloo.ca>
Co-committed-by: Mark Chiu <e26chiu@csclub.uwaterloo.ca>
2023-02-12 17:51:53 -05:00
Shahan Nedadahandeh 743f050ec8 Fix tooltip not being correctly positioned for certain centered graphs (#118)
continuous-integration/drone/push Build is passing Details
Applied the fix used by the WordCloud to all graphs so that tooltips will be properly positioned, even if the graph is centered on the page. Did so by refactoring some tooltip logic in to a common getTooltipPosition used by all graphs.

One thing is that this simplifies all the tooltips to simply follow the mouse, instead of locking onto certain "blocks"/bars like in the grouped bar graph, which is more consitant with all the other tooltips, since we only had this in some cases.

Do we think that having the tooltip be sticky to a certain bar/group is important?

Closes #89

https://fix-tooltip-center-csc-class-profile-staging-snedadah.k8s.csclub.cloud/samplePage/
Co-authored-by: shahanneda <shahan.neda@gmail.com>
Reviewed-on: #118
Reviewed-by: Mark Chiu <e26chiu@csclub.uwaterloo.ca>
2022-12-28 01:54:32 -05:00
Shahan Nedadahandeh 97aa2261f2 Created TooltipWrapper (#88)
continuous-integration/drone/push Build is passing Details
Co-authored-by: shahanneda <shahan.neda@gmail.com>
Reviewed-on: #88
Reviewed-by: Mark Chiu <e26chiu@csclub.uwaterloo.ca>
2022-11-16 21:35:55 -05:00
Mark Chiu 9526f1b0f5 Add BoxPlot component (Closes #6) (#34)
continuous-integration/drone/push Build is passing Details
Done:

- [x] Display boxplot with left and bottom axis in `playground.tsx`
- [x] Add mock data for boxplot
- [x] Hovering over boxplot displays a Tool Tip box.

My version:
![image](/attachments/6c8c4499-a1bd-4434-9230-7117266691ea)

Note:

 - No percentage displayed next to value axis labels.
 - We can add outlier points although it could complicate the graph.
 - Still displays the following error:
 	- Error: Hydration failed because the initial UI does not match what was rendered on the server.
    - Error: Text content does not match server-rendered HTML.
    - Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.

    Staging: https://boxplot-component-csc-class-profile-staging-snedadah.k8s.csclub.cloud/
Co-authored-by: Miniapple8888 <miniapple8888@gmail.com>
Co-authored-by: Emily Chiu <e26chiu@corn-syrup.csclub.uwaterloo.ca>
Co-authored-by: e26chiu <e26chiu@csc.uwaterloo.ca>
Reviewed-on: #34
Reviewed-by: Amy <a258wang@csclub.uwaterloo.ca>
2022-09-03 11:13:58 -04:00