BETA: SciTE v5x & lua Dynamic_include and "Smart" AutoComplete for Vars/UDFs/Abbrevs

[Jos]

  On 10/4/2023 at 12:33 PM, Shark007 said:

Now, interestingly, I used SciTEx86.zip to produce these screenshots

Expand  

Just tried both the current versions of SciTEx86/64.zip and they both produce the exact same result as I posted previously, so the property works fine.
It looks like your ICONs are somewhat smaller than what I see, but that likely is the result of the Screen setting of 2560x1600 on my laptop with 150% scale (both the recommended setting for this device)

Edited October 4, 2023 by Jos

[Shark007]

For reference, my screen is 3840x2160 with 150% recommended but I use 225%

[Shark007]

It really does seem that there is no one size fits all here.

The best we can hope for is to provide the user with easily accessable settings to toy with.

[Jos]

Looking at the code used in SciTE to produce the Toolbar, I see that standard windows calls are used with standard ICONs and those come in only the 2 sizes..... hence the 0/1 option.  
There is an possibility to resize the tile which contains this standard ico to make the clickable space larger, without increasing the size of the ico itself.

This is just toying as I was curious how it was done and played a bit with it when setting the  toolbar.large to a  larger value than the default 32 pixels, it will use that specified size for the tile. EG: Standard toolbar.large=1

 toolbar.large=40

 toolbar.large=60

 

Edited October 4, 2023 by Jos

[Shark007]

Those are interesting results that I cannot reproduce.

[Jos]

  On 10/4/2023 at 2:59 PM, Shark007 said:

Those are interesting results that I cannot reproduce.

Expand  

Correct as it is only available in my "play garden"  

Still considering whether this is worth the extra MOD to maintain and if so, what the best approach would be.

Edited October 4, 2023 by Jos

[Shark007]

I will pay attention to this thread and offer my system(s) as a testbed.

[Jos]

I have updated the SciTEx86/x64.zip files with this test change in SciTE.

• toolbar.large=1  will show the default large Windows icons in an 32 pixel box.
• toolbar.large=20 will show the default large Windows icons in an 32+20-1=51 pixel box.

So the current change is to enlarge the button when toolbar.large > 0 with this formula: 32 - 1 + toolbar.large-value.

Give it a try and see if this would be helpful to keep this way.  

[donnyh13]

Hey Jos, not sure if you want to add any headaches to your project, but I was wondering if this helps you with any ideas for the multi-selection auto complete?

This is a bit rough, adding dyn. incl.  loses the multi selection, but it's more for demonstration.

[snip]

I marked places I modified stuff with "--##BEGIN" & "--##END" comments

expandcollapse popup

line 573    AutoItAutoComplete.lua
    -- List 12: AutoComplete Variables names with the seleted option
    if id == 12 then
        -- insert the text from the selected item.
        editor:BeginUndoAction()
        endPos = editor.CurrentPos
        startPos = editor:WordStartPosition(endPos, true)
--##BEGIN
    --  editor:SetSel(startPos, endPos)
    --##END
        -- this part is used for Abbrev expansion
        if str:find('^%w+%<%->') then
    --##BEGIN   
        editor:SetSel(startPos, endPos)
    --##END 
            self:DebugPrint('AutoComplete Abbreviation:', id, str)
            str = str:match('^(.-)<%->')
            -- Get the total len before expansion
            local beforelen = editor.Length - endPos + startPos
            editor:ReplaceSel(str)
            AutoItTools:Abbreviations(0)
            -- Get the total len after expansion
            local afterlen = editor.Length
            -- do  rangecheck for the added line(s) to check for required Includes
            Check_AddMissingIncludes = 'rangecheck'
            Check_AddMissing_spos = startPos
            Check_AddMissing_epos = startPos + afterlen - beforelen - 1
            Check_Adding_extra_chars = false
        else
            --[[
            -- multiselect logic but not able to insret text at multiple place unless clipboard is used.
            for i = 1, editor.Selections, 1 do
                endPos = editor.SelectionNStart[editor.MainSelection]
                startPos = editor:WordStartPosition(endPos, true)
                --print(editor.MainSelection, startPos, endPos)
                --editor:DeleteRange(startPos, endPos -  startPos)
                editor.SelectionNStart[editor.MainSelection] = startPos
                editor.SelectionNEnd[editor.MainSelection] = endPos
                --editor:InsertText(startPos, str) -- doesn't work
                editor:RotateSelection()
            end
            -- Clipboard is used to replace all Selected Ranges
            --  editor:CopyText(str)
            --  editor:Paste()
            ]]
--##BEGIN
        if (editor.Selections == 1) then

editor:SetSel(startPos, endPos)

        -- Only active range is replaced
            editor:ReplaceSel(str)
            self:DebugPrint('AutoComplete Variable selection:', id, str)
            Check_AddMissingIncludes = 'wordcheck'
            Check_AddMissing_spos = nil
            Check_AddMissing_epos = nil
            Check_Adding_extra_chars = true
        else

local aiSpos = { } -- Array of all current Selection positions
local iSels = editor.Selections -- Number of selections
local iDiff = 0 -- Var for recording the difference between string length already typed, and intended string insertion.

for i = 1, editor.Selections, 1 do -- cycle through all current selections, record their current postions.
aiSpos[i] = editor.CurrentPos

editor:RotateSelection()
end

-- Determine the length of characters that will be added. Take length of total insertion string, minus length of current 
-- typed word.
iDiff = string.len(str) - (aiSpos[1] - editor:WordStartPosition(aiSpos[1]))

            Check_AddMissingIncludes = 'wordcheck'
            Check_AddMissing_spos = nil
            Check_AddMissing_epos = nil
            Check_Adding_extra_chars = true
            
for i = 1, iSels, 1 do -- cycle through selections.

--Select the word already typed for the nth selection.
editor:SetSel(editor:WordStartPosition(aiSpos[i]), aiSpos[i])

editor:ReplaceSel(str) -- replace the text

-- Color inserted text to set the right style
        self:ReLexStyles()
        
        -- if is a Func, add the "(". 
            if editor.StyleAt[editor.CurrentPos - 2] == SCE_AU3_FUNCTION
            or (editor.StyleAt[editor.CurrentPos - 2] >= SCE_AU3_UDF and editor.StyleAt[editor.CurrentPos - 2] <= SCE_AU3_FILEUDF) then
            -- add ( for Funcs/UDFS and trigger the CallTip
            editor:AddText('(')
            
            -- If we are adding a "(", I need to add 1 to my iDiff var. 
            -- I cant add 1 on the first or last rounds, as that would put my positions out of sync. 
if (i ~= 1) and (i ~= iSels) then iDiff = (iDiff + 1) end

            scite.MenuCommand(IDM_SHOWCALLTIP) -- Show ToolTip for Functions
            -- Vars/Keywords: add newline when Enter was pressed on Dropdown ... else add space for Variables and internal keywords
end

-- Cycle through my list of Selection positions, if any come after the current position I am working on, add my difference 
 -- value to keep the correct position.
for j = 1,iSels, 1 do
if (aiSpos[j] > aiSpos[i]) then aiSpos[j] = aiSpos[j] + iDiff end

end

-- update the saved position to the current position after auto completing.
aiSpos[i] = editor.CurrentPos

end

-- Add the selections back so we have multiple again.
for i = 1, iSels, 1 do 
editor:AddSelection(aiSpos[i], aiSpos[i])

end

-- rotate one selection to get back to the original "MainSelection".
editor:RotateSelection()

        end
        
            self:DebugPrint('AutoComplete Variable selection:', id, str)
Check_Adding_extra_chars = false

--##END
        end
        -- Color inserted text to set the right style
        self:ReLexStyles()
        -- check for adding characters after adding the selected Dropdown text
        if not Check_Adding_extra_chars then
            -- Don't add any character for Abbrev's
            -- nothing here

 

Edited March 13 by donnyh13

[Shark007]

setting of 1

setting of 20

setting of 40

@Jos, although not perfect, it certainly gets props for visibility and usability.  Please carry some form of this into the next official release.

I have not tested 32bit yet. I will post of it only if I encounter anomalies.

EDIT: It may be of interest to you, I'd like to explain my use of HiDpi=n

I have recently adopted @argumentum solution for DPI Awareness GUI presentation integration in This Thread which is facilitated by @UEZ solution in This Post because the resulting code is so much cleaner than adding a multiplication factor to all of the coordinates and dimensions as I was doing in the past. I hope these projects see the love they deserve.

Edited October 5, 2023 by Shark007

[Jos]

  On 10/4/2023 at 8:39 PM, donnyh13 said:

This is a bit rough, adding dyn. incl.  loses the multi selection, but it's more for demonstration.

Expand  

I see what you have done working around the losing Multi Sel by saving the positions, then replacing each and then adding them back into the multi selections set.
Have to have a look at that in more detail, but thanks for the idea.  It all still feels strange that AutoComplete isn't working with MultiSelect in the base functionality either.

[Jos]

  On 10/4/2023 at 9:11 PM, Shark007 said:

@Jos, although not perfect, it certainly gets props for visibility and usability.

Expand  

Life's full of compromises, but I am planning to leave this in for now as it also helps me making the Click-and-Hit-rate better.  

[donnyh13]

  On 10/5/2023 at 6:29 AM, Jos said:

It all still feels strange that AutoComplete isn't working with MultiSelect in the base functionality either

Expand  

Ya I know, it's kind of weird.

I did spot this, but I'm guessing it wouldn't be compatible with your method of Auto completing?

Lua Scite API

  Quote

int editor.AutoCMulti -- Change the effect of autocompleting when there are multiple selections.

Expand  

  Quote

SCI_AUTOCSETMULTI(int multi)
SCI_AUTOCGETMULTI → int
When autocompleting with multiple selections present, the autocompleted text can go into just the main selection with SC_MULTIAUTOC_ONCE (0) or into each selection with SC_MULTIAUTOC_EACH (1). The default is SC_MULTIAUTOC_ONCE.

Expand  

 

Edited March 13 by donnyh13

[mLipok]

Something wrong with auto indent.
When I hit ENTER after 1

I get:
 

_Example()
Func _Example()
    Select
    Case 1
        
        
    EndSelect
EndFunc   ;==>_Example

instead expected:

_Example()
Func _Example()
    Select
        Case 1
        
        
    EndSelect
EndFunc   ;==>_Example

 

Edited October 9, 2023 by mLipok

[Jos]

  On 10/9/2023 at 7:33 AM, mLipok said:

Something wrong with auto indent.
When I hit ENTER after 1

 

Expand  

The issue looked like some race condition between scripts, so I have changed the logic somewhat to avoid that in the latest SciTE_changes_Dynamic_Includes_LUA.zip

[Jos]

  On 10/5/2023 at 3:56 PM, donnyh13 said:

Lua Scite API

  Quote

int editor.AutoCMulti -- Change the effect of autocompleting when there are multiple selections.

Expand  

Expand  

When I activate this one via LUA by doing editor.AutoCMulti=1, then the standard SciTE AutoComplete does insert the selected text in all multiselects. Not sure yet how one could configure this as standard behavior when wanted.    

EDIT: Think I found the proper way of doing this so have a play with the latest LUA version in SciTE_changes_Dynamic_Includes_LUA.zip

Edited October 17, 2023 by Jos

[mLipok]

  On 10/12/2023 at 8:17 AM, Jos said:

The issue looked like some race condition between scripts, so I have changed the logic somewhat to avoid that in the latest SciTE_changes_Dynamic_Includes_LUA.zip

Expand  

Confirmed as fixed

[Jos]

  On 10/12/2023 at 10:44 AM, mLipok said:

Confirmed as fixed

Expand  

...Thanks, but I think I have an issue with AutoComplete Abbrevs after the last changes so stay tuned. ....

EDIT: Should be fixed in the latest version (13:06)

Edited October 12, 2023 by Jos

[donnyh13]

Thanks for all of the additions and fixes Jos, I was wondering if you have decided on my feature request from a few pages ago? Sorry if you did repsond, I didn't see it if you did.

  On 9/26/2023 at 4:21 PM, donnyh13 said:

Is it possible to allow ctrl+Backspace to be used (to delete a word at a time) while the AutoComplete drop down is active? Both in single selection and multiple selection?

Expand  

[Jos]

Sorry .... indeed missed that....   need to think/play a bit for that one, but assume it should be possible when that indeed is standard behavior. (Another one I haven't used before  )

EDIT: Uploaded fix for that in the latest SciTE_changes_Dynamic_Includes_LUA.zip

Edited October 16, 2023 by Jos