search

Lootyhoof / stylem

User styles manager for Pale Moon and other Mozilla-style software
GNU General Public License v3.0
22 stars 8 forks source link

Configurable stylesheets doesn't install properly #6

Open Neur0toxine opened 6 years ago

Neur0toxine commented 6 years ago

After installing configurable styles all elements on the page becomes transparent. Styles without configuration button works well. For example, after installing this stylesheet userstyles.org looks like this (you can see transparent header): userstyles.org goes insane And this stylesheet works well: userstyles.org without error

I'm using Pale Moon 28.1.0 x64 on Windows 10 with Stylem 2.2.4. Looks like Stylem just ignoring configurable options, and stylesheets was generated with placeholders instead of selected values.

Lootyhoof commented 6 years ago

Looks to be a JS issue on the site with that particular style:

Unhandled promise rejection undefined 
app.js:1:509452

It does seem to work with other styles, though, like this.

I would say this is more of a browser issue than a Stylem one. Stylem itself doesn't convert those variables - that's something the site does. All Stylem does is run CSS it's provided.

Neur0toxine commented 6 years ago

I checked this in Waterfox 56.2.3. I was able to reproduce bug. But Waterfox supports WebExtensions. So, I installed Stylus 1.4.22 and tested this bug - just to be sure what it's because of incompatibility between browser and userstyles.org. But it's not reproducing with Stylus and Waterfox. Also I installed Stylish 3.1.8 - just to check how it's working in Waterfox. Same as Stylus - I wasn't able to reproduce this bug. Are you sure it's not Stylem bug? I'm asking because currently Stylem is best what can be found for Pale Moon. Old Stylish versions just doesn't compatible with new userstyles.org.

Lootyhoof commented 5 years ago

Sorry for the delay. This has been partially mitigated in v2.2.5 (released just now), in that these styles will install correctly, but only use the default settings. Making them fully work would require large rewrites that I can't justify at this time.