Dark mode - For Test

[Incpi]

Hello All,

This will be the pull request for the dark mode feature. I would greatly appreciate it if you could try it out and provide your valuable feedback.

I am writing custom lib based on fomantic-Ui (works with this project only). Small preview:

• this will fix issue of cpi helper in dark mode.

Make a separate branch if needed.

Thank you. Omkar

[dbeck121]

Hi,

sorry that you have to wait for so long. I will check this weekend!

[dbeck121]

@incpi

can you give a bit background:

• Does it work standalone of in combination with your dark cpi plugin? It feels like the popup with settings work now in dark mode but for cpi itself, we need your plugin right?
• Together with your plugin, messagesidabar does not get dark

In general: works prety good so far.

[Incpi]

Hello @dbeck121,

I'm altering it as I discovered it. Since I wrote my custom CSS (not perfect at all). I also included small dev notes (readme.md) in each lib that I make changes as disclaimer/ warning to be aware of impact of changes in future.

• Indeed, it is with the Dark CPI plugin.
• If someone wants to do a mismatch, such as bright with dark cpi helper, I have included a popup setting. only switching allowed in light mode, No dark mode switching by design.
• with plugin, message sidebar does not get dark. Could you assist with the screenshot? My own is operational. with this branch.

If you need more info, let me know. Thanks Omkar

[fippu82]

Hi @incpi,

I have pulled your changes and while using CPI Helper with Inline Trace, I encountered a bug: After deactivating inline trace (using the button in the sidebar), it is still opening the log popup when clicking on a step that was part of the trace.

Could you please check this as it works in the currently published version. Thanks!

[Incpi]

Actually, @dbeck121 / @fippu82.

It doesn't affect the current version because I made a mistake in the 3.15 version. If you plan to release version 15, kindly include this roughly every 15 lines. I appreciate you informing me of this. I completely forgot about it. I fixed the last commit in this PR.

Regards

[fippu82]

@incpi Thank you. But what do you mean by "kindly include this roughly every 15 lines."? I don't get you.

[Incpi]

@incpi Thank you. But what do you mean by "kindly include this roughly every 15 lines."? I don't get you.

This fix, or last commit from this PR

for 3.15 version. If any planned for release, we need to add these changes there. So no impact in current dev version