[PR #2892] [CLOSED] Add support for solid cursor blinking style #25138

Open
opened 2026-01-31 09:07:31 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2892
Author: @skyline75489
Created: 9/25/2019
Status: Closed

Base: masterHead: feature/cursor-blinking-style


📝 Commits (1)

  • 45af95b Add support for solid cursor blinking style

📊 Changes

8 files changed (+134 additions, -16 deletions)

View changed files

📝 src/cascadia/TerminalApp/Profile.cpp (+54 -0)
📝 src/cascadia/TerminalApp/Profile.h (+3 -0)
📝 src/cascadia/TerminalControl/TermControl.cpp (+53 -16)
📝 src/cascadia/TerminalControl/TermControl.h (+2 -0)
📝 src/cascadia/TerminalSettings/ICoreSettings.idl (+7 -0)
📝 src/cascadia/TerminalSettings/TerminalSettings.cpp (+10 -0)
📝 src/cascadia/TerminalSettings/terminalsettings.h (+3 -0)
📝 src/cascadia/UnitTests_TerminalCore/MockTermSettings.h (+2 -0)

📄 Description

Summary of the Pull Request

Added cursor blinking style. Mainly for solid cursor blinking style which simply disables binking.

References

PR Checklist

Detailed Description of the Pull Request / Additional comments

The idea is from #1379 and VSCode. VSCode has several cursor blinking style, including blink, solid, and other animated styles.

Validation Steps Performed

Add configurationi to profile and see how it looks like:

{
    "cursorShape": "filledBox",
    "cursorBlinking": "solid",
}

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/2892 **Author:** [@skyline75489](https://github.com/skyline75489) **Created:** 9/25/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `feature/cursor-blinking-style` --- ### 📝 Commits (1) - [`45af95b`](https://github.com/microsoft/terminal/commit/45af95b05842b566120b248f974746cd9e59b053) Add support for solid cursor blinking style ### 📊 Changes **8 files changed** (+134 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalApp/Profile.cpp` (+54 -0) 📝 `src/cascadia/TerminalApp/Profile.h` (+3 -0) 📝 `src/cascadia/TerminalControl/TermControl.cpp` (+53 -16) 📝 `src/cascadia/TerminalControl/TermControl.h` (+2 -0) 📝 `src/cascadia/TerminalSettings/ICoreSettings.idl` (+7 -0) 📝 `src/cascadia/TerminalSettings/TerminalSettings.cpp` (+10 -0) 📝 `src/cascadia/TerminalSettings/terminalsettings.h` (+3 -0) 📝 `src/cascadia/UnitTests_TerminalCore/MockTermSettings.h` (+2 -0) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Added cursor blinking style. Mainly for solid cursor blinking style which simply disables binking. ## References ## PR Checklist * [X] Closes #1379 * [X] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [X] Requires documentation to be updated * [X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments The idea is from #1379 and VSCode. VSCode has several cursor blinking style, including `blink`, `solid`, and other animated styles. <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed Add configurationi to profile and see how it looks like: ```json { "cursorShape": "filledBox", "cursorBlinking": "solid", } ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:07:31 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#25138