pulsar-edit / pulsar

A Community-led Hyper-Hackable Text Editor
https://pulsar-edit.dev
Other
3.09k stars 131 forks source link

Window Transparency Setting Messes with Window Size, Style, and Randomly Remains Opaque #1046

Open D-Maxwell opened 6 days ago

D-Maxwell commented 6 days ago

Thanks in advance for your bug report!

What happened?

After noticing that #948 had been considered for implementation, I couldn't help but upgrade to the latest release and enjoy an acrylic or even a plain blurred background for this great IDE ; to my surprise, I was met with quite a few annoyances, some of which looked terrible, and others outright preventing any use of the IDE as it remained in a collapsed state, unable to be focused, along an empty and tiny preview window from the taskbar entry. (this may or may not happen after disabling Window Transparency, postponing the restart, and enabling it once more, before then restarting, although I am now unable to reproduce this last issue)

Pulsar version

1.118.0

Which OS does this happen on?

🪟 Windows

OS details

Windows 10 Pro 22H2 19045.2788

Which CPU architecture are you running this on?

x86_64/AMD64

What steps are needed to reproduce this?

As seen in this first picture, the document's size does not match that of the window. I have also found it impossible to resize the window by its edges, although it does work through AltSnap (a Window Management Script). In such a scenario, the maximise button has no effect either. image

Maximising the window through AltSnap --given the inefficiency of the intended button-- will either yield a black background in place of a transparent view --a screenshot for which I lack--, or will result in the title bar reverting to some XP-esque aesthetic. image

As previously mentioned, at times, the window will spawn without respect to its last set size, and will default to the minimum window size ; which, in conjunction to the document not filling the whole window for some reason, leads to a purely empty window, unless manually expanded ; once again, through the use of a script, as the edges refuse to be considered as such upon hovering. image

Additional Information:

I am quite disappointed seeing this feature that I for one had greatly anticipated, and I would love for it to become usable.

D-Maxwell commented 6 days ago

image After some amount of styling, I could get it to look somewhat good ; that being said, resizing the window and opening the editor remain quite painful.

confused-Techie commented 5 days ago

@D-Maxwell While I appreciate the issue, this is very much known.

We tried to get this across in the setting description of Allows editor windows to be see-through. When this setting is enabled, UI themes and user stylesheets can use background colors with an alpha channel to make editor windows translucent. Takes effect after a restart of Pulsar.. Which the important part to note is "UI themes and user stylesheets can use...".

We are aware that just enabling this doesn't really look that great. There were attempts to make this feature usable on our default provided themes, but doing so was much much easier said than done. Leaving us to essentially offer the ability for "community UI themes and user stylesheets" to utilize this feature.

We know you wanted this feature, so we wanted to make sure we could support you and the rest of the community in having it, even if we don't have the time ourselves to make a stylesheet that makes it perfect.

Considering the time you spent working on your user stylesheet to make this work, I'd highly recommend that you could publish those changes as a community UI theme for everyone else, or even if you don't want to, we could look at using pulsar-cooperative to manage the package while getting the changes you've made to make this available to everyone.

But otherwise I apologize if this was an underwhelming change, but our goal in the PR that was merged was to make this behavior possible rather than take advantage of it completely. While that may or may not change in the future, I'd love to see someone from the community be able to turn it into something truly great, now that it is possible at all.

(I'd like to note I don't speak for DeeDeeG who authored the changes themselves, but was communicating with them through the difficulties in attempting to utilize these features in built in themes, and may have supported the current course of action in simply making the feature possible.)

savetheclocktower commented 5 days ago

I'll admit I was surprised to see the difference between the implied dimensions of the window (as indicated by the title bar and placement of the close/minimize/maximize buttons) and the dimensions that we actually occupy. Was that a known issue?

D-Maxwell commented 4 days ago

I'll admit I was surprised to see the difference between the implied dimensions of the window (as indicated by the title bar and placement of the close/minimize/maximize buttons) and the dimensions that we actually occupy. Was that a known issue?

@confused-Techie seems to have focused on exactly the one thing... that I had been expecting. At no point had I expected Pulsar to incorporate stylings that made use of the window's ability to render transparently. The initial reason I reported this issue --the same as for why i believe its existence was required-- had all to do with every other issue I've (actually) mentioned. Again, at no point was I expecting a ready made theme to be bundled and enabled upon toggling on the setting. I knew I would have to be spending an hour or two getting familiar and experimenting with Atom's inner HTML layout, in order to create my own theme taking advantage of this newly merged feature.

I would certainly prefer for you to focus on the other issues this seems to have brought --and that I have reported above--, as I reckon you must have misunderstood me by expecting my own expectations to be such that Pulsar would automatically "look good" with this option on.

D-Maxwell commented 4 days ago

Considering the time you spent working on your user stylesheet to make this work, I'd highly recommend that you could publish those changes as a community UI theme for everyone else, or even if you don't want to, we could look at using pulsar-cooperative to manage the package while getting the changes you've made to make this available to everyone.

But otherwise I apologize if this was an underwhelming change, but our goal in the PR that was merged was to make this behavior possible rather than take advantage of it completely. While that may or may not change in the future, I'd love to see someone from the community be able to turn it into something truly great, now that it is possible at all.

I had indeed thought of releasing my humble --somewhat messy and hastily put together-- stylesheet, although I have never done that in the past. I may consider getting to it once I am done covering all use cases (as for some reason some panels are unaffected by the global rules I have defined, such as the Settings pane), and once I finish tidying it up.