616-fix-tooltip #621

Merged

b24johka merged 2 commits from 616-fix-tooltip into team_1_week_3

2026-04-28 12:35:47 +00:00

b24erika commented

2026-04-28 11:10:23 +00:00

Collaborator

Added tooltip to the new add view function (file), there was also an missing config for "TemperatureWeekView" which was added as well.

CSS for tooltip is not merged at this moment (another pull-request), currently tooltips will be displayed in an "incorrect" way.

Added tooltip to the new add view function (file), there was also an missing config for "TemperatureWeekView" which was added as well. CSS for tooltip is not merged at this moment (another pull-request), currently tooltips will be displayed in an "incorrect" way.

b24erika added the

frontEnd

Team 1

needs review

labels

2026-04-28 11:10:23 +00:00

b24erika self-assigned this

2026-04-28 11:10:23 +00:00

b24erika added 2 commits

2026-04-28 11:10:24 +00:00

added tooltip config to new add function 7292cdb984

added tooltip "week view" 5c0a96f95e

There was a view missing in tooltip config.

b24erika referenced this pull request

2026-04-28 11:12:51 +00:00

fix tooltip not displaying #616

a24timsv requested review from a24hamrh

2026-04-28 11:30:46 +00:00

a24hamrh commented

2026-04-28 11:55:39 +00:00

Collaborator

Review on #616

Whats done

Tested on Windows 11 using Firefox

App starts
- Made a TemperatureWeekView and checked the tooltip. A text "Displays the average temperature for the week" appeared

Inspected following files useAddNewView.js, tooltipConfig.js

Both
- Indentation/code styling
  - Indentation is ok
- Naming convention
  - Naming convention is followed
- General code quality
  - The general code quality is good.

Conclusion

The app runs and a tooltip for TemperatureWeekView exists and works.

# Review on #616 ## Whats done ### Tested on Windows 11 using Firefox + App starts + Made a TemperatureWeekView and checked the tooltip. A text "Displays the average temperature for the week" appeared ### Inspected following files useAddNewView.js, tooltipConfig.js + Both + Indentation/code styling + Indentation is ok + Naming convention + Naming convention is followed + General code quality + The general code quality is good. ## Conclusion + The app runs and a tooltip for TemperatureWeekView exists and works.