Add comment explaining the URL numbering

52b0e7db · Martin Tobias Holmedahl Sandsmark · Kurt Hindenburg · 88f2462e · 52b0e7db
Commit 52b0e7db authored Oct 03, 2020 by Martin Tobias Holmedahl Sandsmark Committed by Kurt Hindenburg Oct 06, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

src/filterHotSpots/FilterChain.cpp src/filterHotSpots/FilterChain.cpp +5 -1

No files found.
--- a/src/filterHotSpots/FilterChain.cpp
+++ b/src/filterHotSpots/FilterChain.cpp
@@ -256,7 +256,11 @@ void FilterChain::paint(TerminalDisplay* td, QPainter& painter)
    int urlNumInc;

    // TODO: Remove _reverseUrllHints from TerminalDisplay.
-    if (_reverseUrlHints) { // TODO: Access reverseUrlHints from the profile, here.
+    // TODO: Access reverseUrlHints from the profile, here.
+    if (_reverseUrlHints) {
+        // The URL hint numbering should be 'physically' increasing on the
+        // keyboard, so they start at one and end up on 10 (on 0, or at least
+        // they used to, but it seems to have changed, I'll fix that later).
        urlNumber = count(HotSpot::Link);
        urlNumInc = -1;
    } else {