Mica Alt

[Andarwinux]

Description of the new feature/enhancement

Add Mica Alt support

Proposed technical implementation details (optional)

[zadjii-msft]

Eh, sure.

Hardest part would be changing the existing $theme.$window.useMica property in the settings from a straight boolean to one that also accepts a string enum. (ala bellStyle and BellStyle)

[mews6]

Is there anybody working on this? I think i could take a look at it.

[zadjii-msft]

@mews6 go for it! Lemme know if you need any help.

[BretasArthur1]

@mews6 u still working on this issue?

[mews6]

@mews6 u still working on this issue?

Yes, i am. I have been scouting the codebase to check where should i be changing stuff because this is my first time contributing to the project. I'm trying to formulate a good briefing of the changes to be done before i change anything in the code itself and there are already a few things I'm probably going to ask about, but I'm trying to check if there are any obvious responses to my questions before getting back here and asking those questions, if that makes any sense. Sorry if i have taken a bit longer for that, I'm just trying to get accustomed.

I'm currently trying to understand how the settings mentioned by @zadjii-msft work. Just reading documentation (mostly here) on how those terminal settings work and checking where Mica is mentioned in the repo (Which you can check here). I could certainly use a bit of guidance for where i should be changing stuff. It seems that useMica is declared as a function in

terminal/src/cascadia/WindowsTerminal/IslandWindow.cpp

Line 1934 in 56cfb77

void IslandWindow::UseMica(const bool newValue, const double /*titlebarOpacity*/)

and

terminal/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp

Line 1176 in 56cfb77

void NonClientIslandWindow::UseMica(const bool newValue, const double titlebarOpacity)

Among other places.

I'm not still sure if we should be changing both definitions, if there is anything else to be changed or if there are any other considerations i should have regarding this problem. That's the reason i haven't really talked much here, but rest assured i'm working on it and i'm open to feedback.

[BretasArthur1]

I think that u r in the perfect way to solve it! It's my first time here to, and if u wan't to discuss about it, I will be glad to help... I have some ideas to implement to...

[geethika870]

hey can i work on this!

[mews6]

Hmm, for some reason this issue is gathering quite a bit of attention

@zadjii-msft , what should we do about this?

[typenoob]

I think you could just specify mica alt theme by giving DWMSBT_TABBEDWINDOW value to const attribute.

terminal/src/cascadia/WindowsTerminal/IslandWindow.cpp

Line 1934 in 56cfb77

void IslandWindow::UseMica(const bool newValue, const double /*titlebarOpacity*/)

Following code block has called the function we modified.

terminal/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp

Line 1176 in 56cfb77

void NonClientIslandWindow::UseMica(const bool newValue, const double titlebarOpacity)

@mews6 u still working on this issue?

Yes, i am. I have been scouting the codebase to check where should i be changing stuff because this is my first time contributing to the project. I'm trying to formulate a good briefing of the changes to be done before i change anything in the code itself and there are already a few things I'm probably going to ask about, but I'm trying to check if there are any obvious responses to my questions before getting back here and asking those questions, if that makes any sense. Sorry if i have taken a bit longer for that, I'm just trying to get accustomed.

I'm currently trying to understand how the settings mentioned by @zadjii-msft work. Just reading documentation (mostly here) on how those terminal settings work and checking where Mica is mentioned in the repo (Which you can check here). I could certainly use a bit of guidance for where i should be changing stuff. It seems that useMica is declared as a function in

terminal/src/cascadia/WindowsTerminal/IslandWindow.cpp

Line 1934 in 56cfb77

void IslandWindow::UseMica(const bool newValue, const double /*titlebarOpacity*/)

and

terminal/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp

Line 1176 in 56cfb77

void NonClientIslandWindow::UseMica(const bool newValue, const double titlebarOpacity)

Among other places.
I'm not still sure if we should be changing both definitions, if there is anything else to be changed or if there are any other considerations i should have regarding this problem. That's the reason i haven't really talked much here, but rest assured i'm working on it and i'm open to feedback.

[the-sky7]

Hi @zadjii-msft, if this issue has been resolved then please close/mark it done

[DHowett]

Hi zadjii-msft, if this issue has been resolved then please close/mark it done

@the-sky7 Having fully read and understood the comment thread, do you have any reason to believe that this issue is resolved?