clutcher / bh

Issue tracker for Better Highlights Intellij IDEA plugin
7 stars 0 forks source link

Quad slash comments //// do not work #66

Closed alexhorner closed 10 months ago

alexhorner commented 1 year ago

Other people in my team use a plugin for VS. I am configuring Better Highlights to match their configuration for use within Rider.

When using a quadruple comment like so:

////this
////is
////dead
////code

Their plugin will grey out and strike though the code. Configuring // in Better Highlights effects all comments. Configuring //// affects no comments. Additionally, configuing either // or //// shows a blank entry in the colour editor.

clutcher commented 1 year ago

@alexhorner Using //// allows to achieve desired behavior.

image

As of showing blank - I'll check if it could be fixed without UI rewriting. Also I'll check behavior of // and //// not sure that it is working as I would expect it.

alexhorner commented 1 year ago

@clutcher image image image

Can confirm it does actually work... Not sure why it didn't before, because it certainly didn't when I opened this issue. Oh well, the blank display thing is certainly still an issue for sure, please see above,

Thanks!

clutcher commented 1 year ago

@alexhorner Great that it works.

As for blank -> it is expected behavior for intellij (( They are using // as a separator for sections, so to see "!" under comments is generated string "Comment//!" . And unfortunatelly I can't do anything with it without fully rewriting UI((

As for using //// instead of // - it is also bug, but I'm not sure that I would fix it. It has same root cause as #12 and fixing it will lead to great performance degradation. I'll think more about it after rewriting UI, maybe will find another way to fix that without significant impact on performance.

P.S. I have a plans to fully redo UI/UX and stop using default intellij settings, so I would be able to fix this one and few other issues. Hopefully I would do it in next few months.

clutcher commented 10 months ago

@alexhorner After latest release with new UI/UX I will close this ticket. Issue with using //// instead of // I will move into ticket #12 .