533-config-tooltip #569

Closed
b24erika wants to merge 0 commits from 533-config-tooltip into team_1_week_3
Collaborator

The config for tooltip is implemented as well as added to the functionality in dashboard.

The config for tooltip is implemented as well as added to the functionality in dashboard.
all current views now have their own output when hovering the tooltip symbol.
Author
Collaborator

Some tooltips are displayed "outside" of the window, this is a separate issue and not related to this one.

This link shows the issue as well as the functionality.

Some tooltips are displayed "outside" of the window, this is a separate issue and not related to this one. This [link](https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/533#issuecomment-5539) shows the issue as well as the functionality.
Collaborator

Review on #533

Whats done

Reviewed #533

Tested on Windows 11 using Firefox

  • App starts
    • Tested adding different views and hovering over the tooltip icon. A text related to the view was shown.

Inspected following files: Dashboard.js, SaveGrid.js, toolTipConfig.js

  • all inspected files.
    • Indentation/code styling
      • Indentation is ok
    • Naming convention
      • Naming convention is followed
    • Comments
      • Comments are good.
    • General code quality
      • The general code quality is good

Conclusion

  • The code executes and everything works as intended except that sometimes the tootips are displayed "outside" of the window, which is mentioned earlier.
# Review on #533 ## Whats done Reviewed #533 ### Tested on Windows 11 using Firefox + App starts + Tested adding different views and hovering over the tooltip icon. A text related to the view was shown. ### Inspected following files: Dashboard.js, SaveGrid.js, toolTipConfig.js + all inspected files. + Indentation/code styling + Indentation is ok + Naming convention + Naming convention is followed + Comments + Comments are good. + General code quality + The general code quality is good ## Conclusion + The code executes and everything works as intended except that sometimes the tootips are displayed "outside" of the window, which is mentioned earlier.
Collaborator

Review has been conducted in a good fashion, i will merge this manually.

Review has been conducted in a good fashion, i will merge this manually.
b24johka closed this pull request 2026-04-27 10:14:28 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!569
No description provided.