Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Border issues in nightly versus release (W11) #5645

Open
4 tasks done
hemirt opened this issue Oct 13, 2024 · 8 comments
Open
4 tasks done

Border issues in nightly versus release (W11) #5645

hemirt opened this issue Oct 13, 2024 · 8 comments

Comments

@hemirt
Copy link
Contributor

hemirt commented Oct 13, 2024

Checklist

  • I'm reporting a problem with Chatterino
  • I've verified that I'm running the most recent nightly build or stable release
  • I've looked for my problem on the wiki
  • I've searched the issues and pull requests for similar looking reports

Describe your issue

If Windows 11 users could confirm how it appears for them, that would be nice, since this could just be some weird W11 setting combination.

windows 11 has these stupid borders around everything, they also change colors when window is selected or not (also whether its pinned to the side or not)

  • some apps do NOT change these colors when active vs inactive (this would be preferred for me, would be nice to be a setting)

in stable it is that way, stays the same as active border whether the window is active or not
in nightly this is no longer that way, and it switches between inactive and active border states

the main issue is now that there is a small margin around chatterino nightly which looks bad, also the input box is smaller (for some reason the font in the input box too)

both unselected:
{21051AE4-9F09-489D-8154-5F2A33170B81}

stable selected:
{C8DD56DB-2334-4F64-AC07-F25C156A2B57}

nightly selected:
{92229976-93CA-4D4E-9112-6EAE8F0F4442}

as you can see, there is something off about the borders in nightly version, on windows 11

what can be problematic about this issue

  • windows has like several settings affecting borders, border accents, colors, states, pin states, etc, so the way people see this can be different for everyone, so this may just be affecting a certain combination of settings that I have, so if anyone can also confirm if they see the same issues, that would be nice
    • that said, the chat input box should be the same (i.e. not filling up to the borders) for each user, if anyone could confirm image here I zoomed in (and selected the nightly version, so that both versions have the same active border color, stable is in the top right back, whilst nightly is in front bottom left)

(I tried to disable windows scaling, and restart apps, still same issue, so atleast it shouldnt be specific to scaling issues)

Screenshots

No response

OS and Chatterino Version

Chatterino 2.5.1 (commit eafcb941) built with Qt 6.5.0, MSVC 193833135
Running on Windows 11 Version 24H2, kernel: 10.0.26120

Chatterino Nightly 2.5.1 (commit 64864a09) built on 2024-10-12 with Qt 6.7.1, MSVC 194134120, Crashpad
Running on Windows 11 Version 24H2, kernel: 10.0.26120

@hemirt hemirt added the issue-report An issue reported by a user. label Oct 13, 2024
@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

If anyone can confirm that these issues exist, I can see that the Qt version changed, so that might give some directions.

@pajlada pajlada added OS: Windows needs testing and removed issue-report An issue reported by a user. labels Oct 13, 2024
@Nerixyz
Copy link
Contributor

Nerixyz commented Oct 13, 2024

Can you try compiling Chatterino with the following line as auto addBorders = isMaximized (without the win11 check)?

auto addBorders = isMaximized || isWindows11OrGreater();

@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

Can you try compiling Chatterino with the following line as auto addBorders = isMaximized (without the win11 check)?

auto addBorders = isMaximized || isWindows11OrGreater();

trying out

@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

{78B56B04-4082-4F24-985A-17F720F8699A} as you said with = isMaximized;, in the normal nonmaximized window now the border is not changing and is acting as it was before, i.e. always active state, i.e. same as the stable version

there is still a gap, but smaller than what is there without this change ->
{BE6B20CC-657E-442C-A41E-BB5CD0A43E6E}
{7C62112F-C41B-4EDE-96AE-F264D593CA15}

(last two images are without the change, and the very first image in this comment is with the change)

sorry for using black as border color 😅 it makes it a bit difficult to see

@Nerixyz
Copy link
Contributor

Nerixyz commented Oct 13, 2024

(I'm on Windows 10). We're "manually" drawing a border currently:

// Draw a border that's exactly 1px wide
//
// There is a bug where the border can get <dpr>px wide while dragging.
// this "fixes" itself when deselecting the window.
auto dpr = this->devicePixelRatio();
if (dpr != 1)
{
painter.setTransform(QTransform::fromScale(1 / dpr, 1 / dpr));
}
painter.fillRect(1, 1, this->realBounds_.width() - 2,
this->realBounds_.height() - 2, bg);

... which is why we're setting margins of 1 (device independent pixels, not real pixels) [are you on a High-DPI screen?] We can't really have a 1px border since Qt uses integers for dp.

this->ui_.layoutBase->setContentsMargins(1, 0, 1, 1);

On Windows 10, I find it nice to have the border change when selected/deselected. Since I'm not familiar with Windows 11, I don't know what the default behavior is.

@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

If anyone can confirm that these issues exist, I can see that the Qt version changed, so that might give some directions.

did try this out with qt 6.5.3., the input box gap is still there and the active/inactive things works the same as I mentioned when changing that line as you said, therefore I don't think qt version has anything to do with it

@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

Not sure what you mean by high dpi screen, but if you do mean windows scaling then yes, i have 150 % (with res: 2560x1440), I changed scaling to 100% for these images
without change:
{4C5DF48F-11B2-4E0A-B33D-69207A12DBB0}
{81DF074F-EEA5-48F8-A649-8A44E544C618}
(do still bear in mind that the black color is my accent color)

with change:
{E427DD49-F092-41F7-B71A-FD9500717A89}

the gap persists, the active/inactive behaviour is as I mentioned before, I myself prefer the always active for chatterino, it is just a bit too jarring to see the border accent change - since I would prefer no border, alas, windows 11 deems that border must be, then border must be, no way to remove them.

@hemirt
Copy link
Contributor Author

hemirt commented Oct 13, 2024

To summarize then:

  1. So I guess the main issue I have is that the input box is not filled fully to the sides of the window (i dont like it this way because now it looks like a rectangle box is cut off by the rounded window corner.

  2. And the fact that this might be a preferred setting, as you said you like the select/deselect thing in windows 10, whilst in windows 11 I dont like this (I think its due to the fact that borders exist even for deselected windows, I don't think windows 10 did this.).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants