Looking for feedback on final Quickscreen theme tags

[Dook]

Hi folks!

I've been working on adding new theme tags that display the user's quicksettings. Previously, it was possible to obtain the name of the setting a user had put on the quickscreen using %St(qs top) %St(qs left) etc, however it was impossible to get the secondary information like what the default quickscreen can show.

As I'm currently cleaning up and preparing to upload the code for review, I wanted to get some feedback in terms of the chosen tags and how I've implemented the features.

I've added two tags for each Quicksetting direction, so eight in total. For example, the left quicksetting's tags would be %QL and %Ql (lowercase L). The %QL tag displays the name of setting that has been attached to the left button, while the %Ql lowercase tag displays the output of that setting.

I'm wondering what people's thoughts are on this arrangement? Are there any improvements or changes I can make? I do feel that the %QL tag could be seen as redundant considering this information is already accessible with another tag, it does add 40 lines of code to the skin parser which does appear to be a bit large compared to a lot of tags. However having both %QL and %Ql tags provides people with a convention to follow, as well as making the feature easier to access.

[chronicallyoffline]

I think it's a great addition. Slightly adjacent question related to the quickscreen - is there currently a way to map a quickscreen option to the scrollwheel for changing values?

I'd personally be really interested in the ability to map "Brightness" to a single Qs entry, click it, then be able to adjust using the wheel, rather than clicking it 10-20 times for the desired level.

[Dook]

I think a way of changing the setting mapped to the scrollwheel would be an interesting feature. Having it change based on the quickscreen settings though might be a bit too convoluted.