postman-app-support icon indicating copy to clipboard operation
postman-app-support copied to clipboard

When part of commented out Json Raw body is uncommented it fully gets uppercased

Open emilianopenedo opened this issue 5 years ago • 14 comments
trafficstars

Describe the bug When part of commented out Json Raw body is uncommented it fully gets uppercased

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Json Raw Request Body'
  2. Click on 'Ctrl+k + Ctrl+c' to comment Json block
  3. Click on 'Ctrl+k + Ctrl+u' to uncomment Json block (selected text, more than single cursor position)
  4. See error, the uncommented block converted to uppercase

Expected behavior Keep original Json case unchanged

App information (please complete the following information):

  • App Type Native App
  • Postman Version v7.33.1
  • OS: Microsoft Windows [Version 10.0.19041.330]

emilianopenedo avatar Oct 02 '20 17:10 emilianopenedo

Hey @emilianopenedo

Are you able to provide a short recording of this happening in the app, please?

DannyDainton avatar Oct 05 '20 09:10 DannyDainton

Hey @emilianopenedo,

Thank you for reporting this. We were able to reproduce the bug on Windows. As a workaround, can you please try using Ctrl + / to comment/uncomment blocks?

Cheers 🎉

ritwijsinha avatar Oct 06 '20 12:10 ritwijsinha

Hey @emilianopenedo,

Thank you for reporting this. We were able to reproduce the bug on Windows. As a workaround, can you please try using Ctrl + / to comment/uncomment blocks?

Cheers 🎉

Thank you @ritwijsinha I've tried with Ctrl + / with no success. I't seems that the shortcut mappings are the same as VSCode, then I've found that using Ctrl + ç works with my keyboard distribution (sp intl) and keeps the case as expected.

emilianopenedo avatar Oct 13 '20 21:10 emilianopenedo

This is becoming quite annoying that it still hasn't been fixed. Ctrl + / doesn't work. It just opens settings on my end.

ghost avatar Oct 05 '21 16:10 ghost

Postman developer are not able to fix this issue. Workaround: use shift-alt-mouse click to select slashes and remove

MarkusSeb avatar Jan 21 '22 08:01 MarkusSeb

Found this issue today. I find it very peculiar that this is still an issue.

qfyra avatar May 17 '22 09:05 qfyra

I can see Ctrl + / working, can someone verify if this is still a problem?

giridharvc7 avatar Jan 17 '23 13:01 giridharvc7

I can see ctrl + / working, I guess for postman we have to use 'ctrl + / for comment/uncomment both. As we are developers and most of other code editors use 'ctrl + k , ctrl + c to comment and ctrl + k, ctrl + u to uncomment the code. :)

hammad1-byte avatar Jan 26 '23 09:01 hammad1-byte

Hello! =)

Everything indicates that it is a conflict of shortcuts.

The software needs to differentiate that Ctrl+K+U is for comment only and Ctrl+U is for uppercase text. Not both at the same time.

TiagoSantosDeveloper avatar Feb 08 '23 12:02 TiagoSantosDeveloper

This is still a problem. Some APIs won't recognize the values if they are not normalizing the values internally.

felipedrumond avatar May 16 '23 00:05 felipedrumond

@postman-staff What's the problem?

Either kick Ctrl+K,Ctrl+U (bad for Visual Studio Code users) or kick uppercasing by Ctrl+U! You cannot have both!

aureole82 avatar Aug 31 '23 16:08 aureole82

The amount of times an average developer uncomments is by any measure vastly larger than when they desire to upper case. Even SQL folks started nowadays to make the code NOT SHOUT at us...

Viltersten avatar Sep 24 '23 14:09 Viltersten

I think it depends of the layout of the keyboard. For me for example, it worked with Ctrl + ~ to comment and uncomment.

leandrosa avatar Oct 30 '23 18:10 leandrosa

The issue also occurs on Linux (Pop!_OS 22.04 LTS)

SyedAzeemJaved avatar Apr 18 '24 01:04 SyedAzeemJaved

This is still a problem, after going through the comments I tried Ctrl + / and Ctrl + ~ which didn't work for me on Swedish keyboard. What worked for me is Ctrl + '.

farhanalie avatar May 08 '24 09:05 farhanalie

thank you very much. It works

MarkusSeb avatar May 08 '24 09:05 MarkusSeb