gramps
gramps copied to clipboard
Add forwards and backwards navigation using mouse buttons.
I have added the functionality to go backwards and forwards in NavigationView using the side mouse buttons, just like in a browser.
Good idea.
Rebased.
In testing this, I am getting a signal recursion:
Warning: History: Signal recursion blocked. Signals was : active-changed
This occurs at lines 141, 156 and 173 in the file gramps/gui/displaystate.py.
I cannot see this warning. Are you enabling extra warnings somehow?
The warnings appear in the terminal window. I have no extra warnings or debug options enabled.
I'm testing using the example tree in the person view. I cursor down to add about 5 people to the history and then use the mouse buttons to backwards and forwards. The navigation works but the warnings are concerning.
I was testing it only in the Charts view. Now I tried the person view and I get the warning, but only when I click the forwards and backwards buttons on the mouse very fast. If I hold down Alt + Left I get the same warning. I think the problem is emitting a signal to change the active person before the previous one has finished processing.
I think I have fixed the problem by preventing forwards/backwards navigation while history is locked.
An observation about something that may give a false impression of 'flakeyness' the improved mouse forward/back navigation...
When committing a new object to Gramps via the Object Editors, that new object is not added to the History.
I'm NOT suggesting that the active object focus should shift to the new object after an Add. Effectively, the active object briefly shifts to the new object as the commit occurs, then back to where it was. So maybe the History should push an item into the list?
@carlesalbasboix Thank you. It looks like you fixed the problem. I'll merge this now.