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

Tooltip fixed #30

Merged
merged 3 commits into from
Oct 13, 2020
Merged

Tooltip fixed #30

merged 3 commits into from
Oct 13, 2020

Commits on Oct 12, 2020

  1. Tooltip fixed

    @SahanAmarsha
    SahanAmarsha committed Oct 12, 2020
    Configuration menu
    Copy the full SHA
    2364ffc View commit details
    Browse the repository at this point in the history

Commits on Oct 13, 2020

  1. PR Review: 

    - Use MaterialUI great theming system (makeStyle/useStyle) instead of external (s)CCS
- Use more React composition
- Fix indentation to 2 spaces
- Fix space changes: they might not be ok, but your issue/PR is not about changing spaces
- Move all CustomTooltip logic to a separate JSX file
    @emibcn
    emibcn committed Oct 13, 2020
    Configuration menu
    Copy the full SHA
    f0f10e8 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from emibcn/10-chart-tooltip 

    PR Review:
    @SahanAmarsha
    SahanAmarsha committed Oct 13, 2020
    Configuration menu
    Copy the full SHA
    75f37a7 View commit details
    Browse the repository at this point in the history