Skip to content
Snippets Groups Projects

ENH: Show min-width warning in tour and edit mode

Merged Florian Spreckelsen requested to merge f-min-width-warning into f-boot-tour
1 unresolved thread

Summary

Introduce a warning that is shown in tour and edit mode when viewed on small screens (#194 (closed) ).

Focus

The warning elements and the toggling have been added to tour.js and edit_mode.js.

Test Environment

Simply open and close the tour while reducing the window size and check that the warning appears if and only if the tour is open on small screens. Log in and verify the same for the edit mode.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

  • All automated tests pass
  • Reference related Issues
  • Up-to-date CHANGELOG.md
  • Annotations in code (Gitlab comments)
    • Intent of new code
    • Problems with old code
    • Why this implementation?

Check List for the Reviewer

  • I understand the intent of this MR
  • All automated tests pass
  • Up-to-date CHANGELOG.md
  • The test environment setup works and the intended behavior is reproducible in the test environment
  • In-code documentation and comments are up-to-date.
  • Check: Are there spezifications? Are they satisfied?

For further good practices have a look at our review guidelines.

Edited by Daniel Hornung

Merge request reports

Pipeline #7070 passed with warnings

Pipeline: LinkAhead

#7071

    Pipeline passed with warnings for 61d147c9 on f-min-width-warning

    Approval is optional

    Merged by Daniel HornungDaniel Hornung 4 years ago (Apr 20, 2021 12:40pm UTC)

    Loading

    Pipeline #7072 passed with warnings

    Pipeline: LinkAhead

    #7073

      Pipeline passed with warnings for 7f9ef2ac on f-boot-tour

      Activity

      Filter activity
      • Approvals
      • Assignees & reviewers
      • Comments (from bots)
      • Comments (from users)
      • Commits & branches
      • Edits
      • Labels
      • Lock status
      • Mentions
      • Merge request status
      • Tracking
      Please register or sign in to reply
      Loading