search

microsoft / azuredatastudio

Azure Data Studio is a data management and development tool with connectivity to popular cloud and on-premises databases. Azure Data Studio supports Windows, macOS, and Linux, with immediate capability to connect to Azure SQL and SQL Server. Browse the extension library for more database support options including MySQL, PostgreSQL, and MongoDB.
https://learn.microsoft.com/sql/azure-data-studio
MIT License
7.55k stars 899 forks source link

Intellisense keywords don't respect casing setting. #25752

Open peterrow opened 2 months ago

peterrow commented 2 months ago

Type: Bug

I have Mssql > Format: Keyword Casing set to "lowercase", however when I am writing SQL in a new query and intellisense offers to complete words, e.g. begin, end, else etc... then it does so in UPPERCASE incorrectly.

What's worse is that even if I type the whole keyword in lower case, when I press space after it then it converts it to UPPERCASE which I then have to Ctrl-Z.

Fix the bug such that the above setting is respected.

Azure Data Studio version: azuredatastudio 1.48.1 (8825ec21e67a34ede00e4f02a4afc692eb6cab61, 2024-06-13T21:07:19.253Z) OS version: Windows_NT x64 10.0.22631 Restricted Mode: No Preview Features: Enabled Modes:

System Info |Item|Value| |---|---| |CPUs|13th Gen Intel(R) Core(TM) i7-13700H (20 x 2918)| |GPU Status|2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled| |Load (avg)|undefined| |Memory (System)|31.68GB (2.56GB free)| |Process Argv|| |Screen Reader|no| |VM|0%|
Extensions (6) Extension|Author (truncated)|Version ---|---|--- ssmskeymap|kev|1.1.0 admin-pack|Mic|0.0.2 agent|Mic|0.49.0 dacpac|Mic|1.14.0 import|Mic|1.6.1 profiler|Mic|0.14.0
gfody commented 2 months ago

the setting you want is Mssql > Intelli Sense: Lower Case Suggestions

I'm not sure if that other setting is doing anything.

peterrow commented 2 months ago

@gfody Yep that setting does the trick, thx.

I wish the ADS team would just spend a few releases working on small quality of life fixes and a bits and pieces that are in SSMS then we wouldn't rub up against these kinds of issues, settings that do nothing.

The setting that works shouldn't really exist, the one I changed in opening feedback should be the one that works. Otherwise you could very well end up with settings for generate SQL keyword casing etc...whereas one setting to use lower casing everywhere that ADS outputs SQL would be , I assert, the obvious expectation.