Skip to content
Snippets Groups Projects

Online Help shortcut change + redirect to new docu

Merged Simon Hermanns requested to merge 382-shortcut-ctrl-h-seems-to-be-ambigous into master

Changed the shortcut for online help to F1 so it doesn't collide with the shortcut for hiding controls. Also changed redirection to the old wiki to the new docu page.

Reviewer Checklist

General code quality

  • naming conventions are met (see .clang-tidy for detailed information)
  • no static analyzer warnings in new code parts (e.g., use clang-tidy for checking)

General usability

  • old versions of pet-files are still loadable
  • documentation updated (if applicable)

Only if changes in UI

  • new elements are also saved and loaded from pet-file
  • check if tab order is still correct
  • all new SpinBoxes are promoted
  • new keybindings added to Petrack::keyBindings()

Closes #382 (closed)

Edited by d.kilic

Merge request reports

Pipeline #167022 passed

Pipeline passed for 0794938a on 382-shortcut-ctrl-h-seems-to-be-ambigous

Approved by

Merged by Simon HermannsSimon Hermanns 5 months ago (Oct 17, 2024 11:53am UTC)

Merge details

  • Changes merged into master with 34358ec8 (commits were squashed).
  • Deleted the source branch.

Pipeline #167168 passed

Pipeline passed for 34358ec8 on master

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