Bug 474884 - Opinion Request: Lowercase TODO and other strings
Summary: Opinion Request: Lowercase TODO and other strings
Status: RESOLVED INTENTIONAL
Alias: None
Product: frameworks-syntax-highlighting
Classification: Frameworks and Libraries
Component: syntax (show other bugs)
Version: 5.110.0
Platform: Other Linux
: NOR task
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-09-26 04:02 UTC by electro707
Modified: 2023-10-11 01:00 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description electro707 2023-09-26 04:02:14 UTC 
Hello,

I would like your opinion on the following change before I implement it:

For alerts like TODO, FIXME, etc, I suggest we add the `casesensitive` attribute to allow for lower-case versions of those alerts. I know personally at times I prefer to have lower-case `todo` with my projects.

Let me know what you think! If there aren't any objections, I will make a PR to manage that.

Thanks,
Comment 1 Christoph Cullmann 2023-10-10 16:30:59 UTC 
I am not sure about that, you can have stuff like 'this task will be' or 'we removed a deprecated' or similar sentences in comments and now they get strangely highlighted.

I think it is important that we do that only for upper cased stuff that will not occur in normal comments.

Sorry that I did not react earlier.
Comment 2 Waqar Ahmed 2023-10-11 01:00:56 UTC 
You cqn choose to override it yourself though, locally of course