diff -r 7f643d41464e -r f904d0eef264 eric6/WebBrowser/Bookmarks/BookmarksToolBar.py --- a/eric6/WebBrowser/Bookmarks/BookmarksToolBar.py Wed Jun 17 17:12:21 2020 +0200 +++ b/eric6/WebBrowser/Bookmarks/BookmarksToolBar.py Wed Jun 17 20:18:54 2020 +0200 @@ -135,27 +135,27 @@ @param idx index of the activated bookmark (QModelIndex) """ - assert idx.isValid() - - if self._mouseButton == Qt.XButton1: - self.__mw.currentBrowser().triggerPageAction(QWebEnginePage.Back) - elif self._mouseButton == Qt.XButton2: - self.__mw.currentBrowser().triggerPageAction( - QWebEnginePage.Forward) - elif self._mouseButton == Qt.LeftButton: - if self._keyboardModifiers & Qt.ControlModifier: - self.newTab.emit( - idx.data(BookmarksModel.UrlRole), - idx.data(Qt.DisplayRole)) - elif self._keyboardModifiers & Qt.ShiftModifier: - self.newWindow.emit( - idx.data(BookmarksModel.UrlRole), - idx.data(Qt.DisplayRole)) - else: - self.openUrl.emit( - idx.data(BookmarksModel.UrlRole), - idx.data(Qt.DisplayRole)) - self.__updateVisitCount(idx) + if idx.isValid(): + if self._mouseButton == Qt.XButton1: + self.__mw.currentBrowser().triggerPageAction( + QWebEnginePage.Back) + elif self._mouseButton == Qt.XButton2: + self.__mw.currentBrowser().triggerPageAction( + QWebEnginePage.Forward) + elif self._mouseButton == Qt.LeftButton: + if self._keyboardModifiers & Qt.ControlModifier: + self.newTab.emit( + idx.data(BookmarksModel.UrlRole), + idx.data(Qt.DisplayRole)) + elif self._keyboardModifiers & Qt.ShiftModifier: + self.newWindow.emit( + idx.data(BookmarksModel.UrlRole), + idx.data(Qt.DisplayRole)) + else: + self.openUrl.emit( + idx.data(BookmarksModel.UrlRole), + idx.data(Qt.DisplayRole)) + self.__updateVisitCount(idx) def __openBookmark(self, act): """