Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand/collapse merged events #95

Open
jpnurmi opened this issue Jan 3, 2015 · 3 comments
Open

Expand/collapse merged events #95

jpnurmi opened this issue Jan 3, 2015 · 3 comments

Comments

@jpnurmi
Copy link
Member

jpnurmi commented Jan 3, 2015

It would be better if it was possible to expand/collapse the merged event blocks instead of that stupid tooltip which was thrown in as an easy temporary solution. It might be necessary to switch from QTextBrowser to QPlainTextEdit which should respect QTextBlock visibility attribute. The same mechanism can be then also used for filtering out lines for #93.

@jpnurmi
Copy link
Member Author

jpnurmi commented Jan 3, 2015

PS. There were some good reasons why QTextBrowser was used instead of supposedly lighter QPainTextEdit, but I can't remember the issues...

@jpnurmi
Copy link
Member Author

jpnurmi commented Jan 3, 2015

Preliminary findings for QPlainTextEdit:

  • no smooth pixel-based scrolling :(
  • missing anchorClicked(QUrl) signal
  • does not respect QTextBlockFormat::lineHeight
  • does not set Qt::PointingHandCursor over links
  • missing createStandardContextMenu(QPoint) overload

@jpnurmi
Copy link
Member Author

jpnurmi commented Jan 11, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants