Bug 407119

Summary: Wraparound mode refers to non-existent keyboard shortcut
Product: [Applications] krita Reporter: Karl Ove Hufthammer <karl>
Component: GeneralAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: git master (please specify the git hash!)   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/kde/krita/commit/732f4ae4c0a2963eaf9f8fce08335cd0e9f7f53c Version Fixed In:

Description Karl Ove Hufthammer 2019-05-01 08:03:54 UTC 
SUMMARY
When one enters wraparound mode, the following message is shown:

  Entering Wraparound mode. Press '%1' to leave Wraparound mode.

(where %1 is the keyboard shortcut). But when no keyboard shortcut is defined, the message says:

  Entering Wraparound mode. Press '' to leave Wraparound mode.

(and the '' looks like the " symbol), which is confusing. The second sentence should only be shown when there actually is a keyboard shortcut defined.


STEPS TO REPRODUCE
1. Open an image.
2. Ensure you don’t have a keyboard shortcut for wraparound defined (I didn’t have one by default).
3. Go to ‘View → Wrap Around Mode’.


OBSERVED RESULT
The string ‘Entering Wraparound mode. Press '' to leave Wraparound mode.’ is shown.

EXPECTED RESULT
The string ‘Entering wraparound mode.’ (note the lowercase w) should be shown.

(Or perhaps ‘tile view’ would be a better name than ‘wraparound mode?)
Comment 1 Halla Rempt 2019-05-01 08:32:13 UTC 
Git commit 732f4ae4c0a2963eaf9f8fce08335cd0e9f7f53c by Boudewijn Rempt.
Committed on 01/05/2019 at 08:32.
Pushed by rempt into branch 'master'.

Remove wraparound warning message

Since a) nobody who got confused by wraparound mode ever seems to
have noticed the message and b) we removed the default shortcut
because people got confused too much by it, we might just as well
remove the warning completely. It takes deliberate action to
activate wraparound mode, let's assume that people who do so, know
what they are doing.

M  +0    -6    libs/ui/canvas/kis_canvas_controller.cpp

https://invent.kde.org/kde/krita/commit/732f4ae4c0a2963eaf9f8fce08335cd0e9f7f53c