Skip to content

Conversation

@Roemer
Copy link
Contributor

@Roemer Roemer commented Nov 16, 2021

  • Added a new Tooltip component (so the tooltips are always the same)
  • Added more help texts
  • Fixed build and notes

Check especially the new help texts if they contain the correct information.

Replaces PR #247

@Roemer Roemer mentioned this pull request Nov 16, 2021
import { Tooltip as MuiTooltip } from "@material-ui/core";
import React from "react";

export class Tooltip extends React.Component<any, any> {
Copy link
Member

Choose a reason for hiding this comment

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

make it as functional component
also avoid using any

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I tried to make it a functional component, seems to work.

README.md Outdated

- clone repo
- `npm i`
- `npm run build`
Copy link
Member

Choose a reason for hiding this comment

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

not sure that it's needed to run before local start
why would you need to run it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, works without. I removed it.

@Roemer Roemer force-pushed the feature/help-texts branch from 4b3b630 to ec74879 Compare November 18, 2021 10:38
@Roemer Roemer force-pushed the feature/help-texts branch from ec74879 to 08ef4bb Compare November 18, 2021 10:50
Copy link
Member

@pashidlos pashidlos left a comment

Choose a reason for hiding this comment

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

LGTM!

@pashidlos pashidlos merged commit ee5b5ed into master Nov 18, 2021
@pashidlos pashidlos deleted the feature/help-texts branch November 18, 2021 12:20
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