thatsIch / sublime-rainmeter

Rainmeter Package for Sublime Text 3

Home Page:https://thatsich.github.io/sublime-rainmeter/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update snippets to meet the latest doc

hyrious opened this issue · comments

The snippets were commited 5 years ago and skin options have changed these years.
I would like to see an update according to the latest doc.
In fact, I've collected a list of it at here.
If you decide to use it, make sure to run the process_docs.rb again.

Thanks.

Hey,

thats a nifty tool you created there :-)
I am currently a bit busy, if you want to PR the current state, I can review it quicker
or you have to wait at least a week until I have some free time.

Cheers
Ich

@hyrious

What do you expect from the snippets? They were always meant to prime your measures and meters and not to opt out every potential option.

Maybe only including required or non-optional options is better?

@thatsIch

I was also thinking about what snippets may be more convenient, and also had some try personally (yet not put it on github).

If only including required options, it will be necessary to write another snippets like "tpos (x/y/w/h)" since they are optional. Yet putting these general options in a [StyleGroup] section may be better. Howerer, it comes to the style/patterns of making skins, none of the snippets' business. 🤔

In short, I've tried to make snippets to meet goals as follows:

  • ready-to-use. It's for meters, when snippets are put in, the meter will be visible for users to see.
  • indented. The original snippets have done this, but $0 isn't at the beginning of line.
  • simple or short. Maybe only including required or non-optional options.

I do not use any snippets, so.. I am a bit bummed. Snippets by definition require to be reusable. So they can not be totally generic (so probably only required is no-go). Maybe going by every possible meter out there is the wrong way?

Maybe more useful snippets are the ones like the color snippet.

Or maybe we just need to discuss, what is update-worthy?

Maybe it's unnecessary.