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

Add blinking cursor support #540

Merged
merged 3 commits into from May 9, 2020

Conversation

kklemon
Copy link

@kklemon kklemon commented Jul 8, 2019

Adds support for blinking cursor.

The qmltermwidget submodule is updated to expose a corresponding property (see Swordfish90/qmltermwidget#25).

The blinking cursor option is enabled by default and can be tweaked in the terminal settings tab.

@Swordfish90
Copy link
Owner

Hi @kklemon . This looks very good, thank you. The problem I see with blinking cursor is that it might be a bit heavy on resources (since it's a whole terminal repaint at every second), could you please set it by default to false and update the default profiles to have it disabled (this should avoid breaking backward compatibility)?

@kklemon
Copy link
Author

kklemon commented Jul 20, 2019

That's a valid point but the cursor does only blink if the window is focused and in this case, the user is probably typing anyway which also casts redraws. Also, the 500 ms default blink interval is much longer than the ones of most other effects.

But disabling it by default for backward compatibility is a good point.

@Swordfish90 Swordfish90 merged commit 5fe26ed into Swordfish90:master May 9, 2020
@Swordfish90
Copy link
Owner

Thank you very much and sorry for the very long delay with this pull request.

@iamchriswick
Copy link

iamchriswick commented Mar 17, 2021

Is there support for blinking cursor now? And if so, how do I enable it? 😄

EDIT 1: I can see that there was added support for it on May 29 2020, but that there is no release for at, so will you release v1.1.2?

EDIT 2: Just tried to build , but get the following error:

Fontconfig error: "/etc/fonts/local.conf", line 1: no element found
QQmlApplicationEngine failed to load component
qrc:/main.qml:137:5: Type TerminalContainer unavailable
qrc:/TerminalContainer.qml:23:5: Type PreprocessedTerminal unavailable
qrc:/PreprocessedTerminal.qml:97:9: Cannot assign to non-existent property "blinkingCursor"
Cannot load QML interface

@kklemon
Copy link
Author

kklemon commented Mar 17, 2021

I haven't been involved with the project for quite some time, but indeed the feature is merged into master but with no release yet.

If you want to build it manually, you have to build and install https://github.com/Swordfish90/qmltermwidget from master as well, as the functionality is essentially implemented there.

@andmarti1424
Copy link

I get the same. I have built that qmltermwidget and generated the libqmltermwidget.so. but where I need to place it so it can work? Thanks.

@andmarti1424
Copy link

I answer my self. It goes in /usr/lib/qt/qml/QMLTermWidget/libqmltermwidget.so

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

4 participants