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

Implement showing stderr as the tooltip #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fatalis
Copy link

@fatalis fatalis commented Jun 5, 2018

Optionally as Qt's subset of HTML4

Optionally as Qt's subset of HTML4
@fatalis
Copy link
Author

fatalis commented Jun 6, 2018

Fixing an issue, please don't merge

@fatalis
Copy link
Author

fatalis commented Jun 6, 2018

Nevermind, not related to my code

com.github.zren.commandoutput/contents/ui/lib/ConfigPage.qml:25:2: QML Loader: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
com.github.zren.commandoutput/contents/ui/lib/ConfigCheckBox.qml:11:11: Unable to assign [undefined] to bool

@Zren
Copy link
Owner

Zren commented Jun 6, 2018

There's Kargos that has a specific "syntax" for showing text in a popup.

https://store.kde.org/p/1173112/

I'm a bit worried with abusing the stderr io stream for this. What if we had a mode for showing line 2+ in the tooltip (or popup)?

As for the richtext setting, won't it be Text.AutoText by default?

http://doc.qt.io/qt-5/qml-qtquick-text.html#textFormat-prop

@fatalis
Copy link
Author

fatalis commented Jun 6, 2018

I tried AutoText and it wasn't working with something like <table><tr><td>test</td><td>test2</td></tr></table>. Also, setting it to PlainText makes newlines appear as newlines instead of having to use <br>.

stderr was completely ignored and unused so I figured I'd use it. If they don't want it in the tooltip they can just redirect it, or a setting can be implemented.

@fatalis
Copy link
Author

fatalis commented Jun 6, 2018

Close this if you want, switching to kargos.

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

Successfully merging this pull request may close these issues.

None yet

2 participants