Bug 474884

Summary:	Opinion Request: Lowercase TODO and other strings
Product:	[Frameworks and Libraries] frameworks-syntax-highlighting	Reporter:	electro707 <develop>
Component:	syntax	Assignee:	KWrite Developers <kwrite-bugs-null>
Status:	RESOLVED INTENTIONAL
Severity:	task	CC:	christoph, walter.von.entferndt, waqar.17a
Priority:	NOR
Version First Reported In:	5.110.0
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

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