[NeoChat] [Bug 459997] Ctrl+PageUp/PageDown shortcuts navigate up or down twice

2023-02-17 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=459997

--- Comment #3 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/network/neochat/-/merge_requests/807

-- 
You are receiving this mail because:
You are watching all bug changes.

[NeoChat] [Bug 459997] Ctrl+PageUp/PageDown shortcuts navigate up or down twice

2023-02-20 Thread Tobias Fella
https://bugs.kde.org/show_bug.cgi?id=459997

Tobias Fella  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/netw
   ||ork/neochat/commit/e0f16054
   ||fca51590f05c250cec60a234dad
   ||3ac9e

--- Comment #4 from Tobias Fella  ---
Git commit e0f16054fca51590f05c250cec60a234dad3ac9e by Tobias Fella, on behalf
of Kevin Wolf.
Committed on 20/02/2023 at 17:07.
Pushed by tfella into branch 'master'.

Fix Ctrl+PgUp/PgDn shortcut for room switching

Before this change, Ctrl+PgUp/PgDown didn't select the next room, but
actually moved two rooms at once because the signal handler was called
twice. I'm not entirely sure why this happens (it might be a bug in a
lower layer?), but as we don't want to handle the same event elsewhere,
accepting the event doesn't feel wrong and it appears to fix the issue.
So let's just do that.

M  +2-0src/qml/Page/RoomPage.qml

https://invent.kde.org/network/neochat/commit/e0f16054fca51590f05c250cec60a234dad3ac9e

-- 
You are receiving this mail because:
You are watching all bug changes.

[NeoChat] [Bug 459997] Ctrl+PageUp/PageDown shortcuts navigate up or down twice

2022-11-27 Thread Tobias Fella
https://bugs.kde.org/show_bug.cgi?id=459997

Tobias Fella  changed:

   What|Removed |Added

 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.

[NeoChat] [Bug 459997] Ctrl+PageUp/PageDown shortcuts navigate up or down twice

2022-11-27 Thread Tobias Fella
https://bugs.kde.org/show_bug.cgi?id=459997

--- Comment #1 from Tobias Fella  ---
This looks like a Qt bug; the signal we connect to is emitted twice

-- 
You are receiving this mail because:
You are watching all bug changes.

[NeoChat] [Bug 459997] Ctrl+PageUp/PageDown shortcuts navigate up or down twice

2022-11-27 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=459997

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #2 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/network/neochat/-/merge_requests/676

-- 
You are receiving this mail because:
You are watching all bug changes.