-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Allow the thickness of the bar cursor to be configured #2590
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't look like the DOM renderer picks up these changes yet? It would be good to add a test in the DOM renderer for this as well (DomRendererRowFactory).
Should be good now! Thanks for the review @Tyriar! |
0e4a858
to
79ea49c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm thanks, I tweaked the validation to ensure it's a decimal as well
[email protected] Diff: xtermjs/xterm.js@8341c35...2a9e16b - Include in word separators xtermjs/xterm.js#2583 - Remove unused imports/functions xtermjs/xterm.js#2585 - force alpha to 1 when using background color as inverted foreground color xtermjs/xterm.js#2560 - Fix minimumContrastRatio on dom/truecolor xtermjs/xterm.js#2602 - v4.3.0 xtermjs/xterm.js#2605 - Avoid roundtrip to browser when double-disposing. xtermjs/xterm.js#2616 - Allow the thickness of the bar cursor to be configured xtermjs/xterm.js#2590 - update version of node-pty xtermjs/xterm.js#2621 - Implement hidden in DOM and WebGL renderers xtermjs/xterm.js#2625 - Expose texture atlas as API and use in demo xtermjs/xterm.js#2626 - Webgl v0.4.1 xtermjs/xterm.js#2628 - Add Linode to real world uses xtermjs/xterm.js#2636 - Added Gus to list of xterm real-world users xtermjs/xterm.js#2631 - Remove a large portion of InputHandler's dependency on Terminal xtermjs/xterm.js#2637 - Move back to reseting parser only on RIS xtermjs/xterm.js#2640 - Set glyph fg color based on original bg, not selection xtermjs/xterm.js#2650 - format color value to style '#rrggbbaa' xtermjs/xterm.js#2629 - Use register over add for APIs returning disposables xtermjs/xterm.js#2651 - Standardize how colors helper lib is structured xtermjs/xterm.js#2653 - Fullwidth handling in buffer writes xtermjs/xterm.js#2644 - Target es5 in attach addon xtermjs/xterm.js#2654 Fixes #86194 Fixes #87918 Part of #87456
[email protected] Diff: xtermjs/xterm.js@8341c35...2a9e16b - Include in word separators xtermjs/xterm.js#2583 - Remove unused imports/functions xtermjs/xterm.js#2585 - force alpha to 1 when using background color as inverted foreground color xtermjs/xterm.js#2560 - Fix minimumContrastRatio on dom/truecolor xtermjs/xterm.js#2602 - v4.3.0 xtermjs/xterm.js#2605 - Avoid roundtrip to browser when double-disposing. xtermjs/xterm.js#2616 - Allow the thickness of the bar cursor to be configured xtermjs/xterm.js#2590 - update version of node-pty xtermjs/xterm.js#2621 - Implement hidden in DOM and WebGL renderers xtermjs/xterm.js#2625 - Expose texture atlas as API and use in demo xtermjs/xterm.js#2626 - Webgl v0.4.1 xtermjs/xterm.js#2628 - Add Linode to real world uses xtermjs/xterm.js#2636 - Added Gus to list of xterm real-world users xtermjs/xterm.js#2631 - Remove a large portion of InputHandler's dependency on Terminal xtermjs/xterm.js#2637 - Move back to reseting parser only on RIS xtermjs/xterm.js#2640 - Set glyph fg color based on original bg, not selection xtermjs/xterm.js#2650 - format color value to style '#rrggbbaa' xtermjs/xterm.js#2629 - Use register over add for APIs returning disposables xtermjs/xterm.js#2651 - Standardize how colors helper lib is structured xtermjs/xterm.js#2653 - Fullwidth handling in buffer writes xtermjs/xterm.js#2644 - Target es5 in attach addon xtermjs/xterm.js#2654 Fixes microsoft#86194 Fixes microsoft#87918 Part of microsoft#87456
This setting will allow consumers to change the thickness of the bar cursor style and defaults to the current behavior.
Before (cursorBarWidth = 1):
After (cursorBarWidth = 2):