Skip to content

'opt_all_words' should apply the color to all occurrences of selected text #33

@d0vgan

Description

@d0vgan

At first, thank you for this plugin!
Secondly, I'm not sure whether it is a correct place for suggestions since CudaText contains a newer version of this plugin than the sources available here.

Anyway, my suggestion is to change the line

if not is_sel and opt_all_words:

to

if opt_all_words:

This simple change gives a huge benefit. For example, it allows to select a part of a word - and to apply the selected color to all occurrences of this selected part of the word. Also, it fixes the following misunderstanding: if a whole word is selected, this change allows to apply the selected color to all occurrences of the selected word that exactly matches the option name 'opt_all_words'. Otherwise, without this change, only the currently selected word is colored, thus contradicting the part 'all' in the option name 'opt_all_words'.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions