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.

Sign up for GitHub

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

Jump to bottom

[textinput] Esc,L and Esc,U not finding word boundary #10136

Closed
ferdymercury opened this issue Mar 16, 2022 · 0 comments · Fixed by #10144
Closed

[textinput] Esc,L and Esc,U not finding word boundary #10136

ferdymercury opened this issue Mar 16, 2022 · 0 comments · Fixed by #10144
Assignees
@jalopezg-git
Labels
affects:master bug in:Core Libraries

Comments

@ferdymercury
Copy link
Contributor

Describe the bug

Esc,L and Esc,U are supposed to lower or upper the word, but they seem to apply on the character.

Expected behavior

It applies capitalization on the word.

To Reproduce

  • Write AbCd EfGh
  • Try to apply Esc,L, and Esc,U

Setup

   ------------------------------------------------------------------
  | Welcome to ROOT 6.27/01                        https://root.cern |
  | (c) 1995-2021, The ROOT Team; conception: R. Brun, F. Rademakers |
  | Built for linuxx8664gcc on Jan 12 2022, 10:17:19                 |
  | From heads/master@v6-25-01-2870-gdac9b6398d                      |
  | With c++ (Ubuntu 8.4.0-1ubuntu1~18.04) 8.4.0                     |
  | Try '.help', '.demo', '.license', '.credits', '.quit'/'.q'       |
   ------------------------------------------------------------------

Additional context

#10121
jalopezg-git added a commit to jalopezg-git/root that referenced this issue Mar 16, 2022
@jalopezg-git
[textinput] Fix implementation of the kCmdWordTo{Lower,Upper} edito…
4421056 
…r commands

These commands, binded respectively to `ESC l` and `ESC u`, should
{lower,upper}case the next word; however, only the first character was changed.

Fixes root-project#10136.
@jalopezg-git jalopezg-git mentioned this issue Mar 16, 2022
Merged
2 tasks
jalopezg-git added a commit to jalopezg-git/root that referenced this issue Mar 16, 2022
@jalopezg-git
[textinput] Fix implementation of the kCmdWordTo{Lower,Upper} edito…
8e8d9d5 
…r commands

These commands, bound respectively to `ESC l` and `ESC u`, should {lower,upper}case
the next word; however, only the first character was changed.

Fixes root-project#10136.
jalopezg-git added a commit that referenced this issue Mar 16, 2022
@jalopezg-git
[textinput] Fix implementation of the kCmdWordTo{Lower,Upper} edito…
466d814 
…r commands

These commands, bound respectively to `ESC l` and `ESC u`, should {lower,upper}case
the next word; however, only the first character was changed.

Fixes #10136.
Neel-Shah-29 pushed a commit to Neel-Shah-29/root-1 that referenced this issue Apr 6, 2022
@jalopezg-git
[textinput] Fix implementation of the kCmdWordTo{Lower,Upper} edito…
8ff2cb6 
…r commands

These commands, bound respectively to `ESC l` and `ESC u`, should {lower,upper}case
the next word; however, only the first character was changed.

Fixes root-project#10136.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jalopezg-git jalopezg-git

Labels
affects:master bug in:Core Libraries
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[textinput] Fix implementation of the kCmdWordTo{Lower,Upper} editor commands jalopezg-git/root
2 participants
@ferdymercury @jalopezg-git