Support Qt6

[ahcorde]

No description provided.

[asymingt]

I realize you didn't write this code originally.

However, in the worst case you're going to evaluate N conditionals to get the desired action. Consider doing this in O(1) with a dictionary of function pointers declared on init, which IMO is more readable:

 def __init__(self, context, publish_clock):
    ...
    self.KEY_TO_ACTION = {
        Qt.Key.Key_Space: self._timeline.toggle_play,
        Qt.Key.Key_Home: self._timeline.navigate_start,
        Qt.Key.Key_End: self._handle_end_clicked,
        Qt.Key.Key_Plus self._handle_faster_clicked,
        Qt.Key.Key_Equal: self._handle_faster_clicked,
        Qt.Key.Key_Minus: self._handle_slower_clicked,
        Qt.Key.Key_Left: self._timeline.translate_timeline_left,
        Qt.Key.Key_Right: self._timeline.translate_timeline_right,
        Qt.Key.Key_Up self._handle_zoom_in_clicked,
        Qt.Key.Key_PageUp: self._handle_zoom_in_clicked,
        Qt.Key.Key_Down: self._handle_zoom_out_clicked,
        Qt.Key.Key_PageDown: self._handle_zoom_out_clicked
    }

Then you just call this here:

self.KEY_TO_ACTION.get(event.key(), lambda: None)()

[asymingt]

Consider try - catch with ImportErrror