mouse0270 / lib-themer

A library used to create css variables for use in themes
MIT License
3 stars 2 forks source link

A List of Issues - July 18, 2023 #48

Open StarOfChorusim opened 1 year ago

StarOfChorusim commented 1 year ago

Hiya Mouse! It's awesome to see this module updated to cover some of the previous issues and such folks were having,, but I wanted to pop in and give you a list of things that me and some follow DMs have noticed after playing around for a couple hours.

I'm loving what we're seeing thus far with this module though! It's so close to what I've been asking for, for ages!

mouse0270 commented 1 year ago
StarOfChorusim commented 1 year ago

First off, I wanted to thank you for your swift response, Mouse! I can only imagine all the work you put into your day to day so I appreciate you taking the time out of your day to help where you can.


Before I proceed to responding to your other points, I see that you're on V11. I'm on V10, so it's likely there's some form of System Version discrepancy that's getting in the way. Admittedly, I don't know how much that would effect things.

Now onto each point:

  1. I'll make a note about informing Trusted Players to refresh if the module isn't taking effect for them.
  2. I'll attach my preset for you to take a look at, as well as my module list...which is admittedly quite extensive so it's quite likely that things are heavily conflicting with one another. I won't expect you to look at all of the contingencies on your own, so at some point during my free time this week I'll look into which of my modules could be causing the difference. Modules - StarOfChorusim Module List.txt Starry Slate Preset - https://drive.google.com/file/d/1FFLZ5xOOHiHVZ4lvm6YofftFIDr3S2Zh/view?usp=sharing
  3. Oh no D: That's wild that there's no available documentation provided on those variables, that's very unhelpful if you want to provide clarity. Definitely not your fault. Sadly I'm not someone who can find or provide that either. Hopefully we can find someone who can!
  4. I really only clear my browser cache once every 3 months and only stick to one browser unless something weird happens so this may just be one of those weird anomalies that happens when I put my hands on something. Hardly the first time and why all my friends call me their favorite playtester.
StarOfChorusim commented 1 year ago

So even turning everything else off except my module management stuff and then tried turning off those and it wouldn't change the little hang up I have with the title on that card. It's a smaller issue in my book at least. Just going to have to make sure to let my friends know who were curious too. The only luck I had was adjusting the Deep Purple Color. chrome_59xM3TVksL chrome_4jUWzti4dU

This might be a V10 quirk.

mouse0270 commented 1 year ago

Oh my apologizes, I meant I have fixed the text and it will be included in the next update.

I am unable to identify what is causing the issue with your button though, as I am not experiencing that issue.

  1. Your players shouldn't have to refresh, I will see what is causing this issue, refreshing is just a temporary fix until I can resolve it.

  2. I imported your preset, but I am curious if something imported incorrectly. My chat cards don't look like yours. My windows look like this: image

  3. Yeah, the missing documentation on their css variables makes it quite difficult to give them descriptions. Also they don't appear to be used in consistent type locations. So even when I was going to try and document them, I couldn't make sense of them.

  4. If you are losing your theme, the only thing I can think of is your local browser cache being reset, as I don't really do anything special there. Its just a standard client setting in foundry, and foundry just uses the browser local storage. So if you are clearing that or switching browsers, you would lose the settings. Not really an easy fix the for that. Though, there is a new persistent storage option in v11, so I might look into using that, it will help keep things consistent between players and even allow themes per would as the theme would be stored on the server instead of local browser session

mouse0270 commented 1 year ago

I fixed some of these issues in v1.0.10. I will keep working on them as I can. Sorry for the delay.