Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme-darkmatter #18

Open
olimpiumarius opened this issue Jun 8, 2018 · 9 comments
Open

Theme-darkmatter #18

olimpiumarius opened this issue Jun 8, 2018 · 9 comments

Comments

@olimpiumarius
Copy link

DarkMatter theme is broken....
capture

@Goro2030
Copy link

Goro2030 commented Jun 20, 2018

Yes, urgent attention is needed!! @chron0

@davidc502
Copy link

Does anyone know if this is being looked at or worked?

Best Regards,

@james-hr
Copy link

james-hr commented Aug 2, 2018

Yes, would love to get this issue resolved.
Also, some of the "Save and Apply" buttons do not work. A workaround is to press enter, and then apply the unsaved changes. But would be nice if it works as it did on older versions.

@Goro2030
Copy link

Goro2030 commented Aug 2, 2018 via email

@chron0
Copy link
Member

chron0 commented Aug 2, 2018

nah, I just don't prioritize the precious little time I have to push projects only to get into stupid arguments with openwrt dev crowd. I tried to get it merged but as you can check on the ML they didn't like that my email adress is chrono@foo so it got rejected. And since openwrt/lede still hasn't managed to get current master going on the ar-300m and other NAND devices I'm stuck on an old 17.01 and can't use current master, therefore I won't be able to fix any bugs they've introduced again.

@chron0
Copy link
Member

chron0 commented Aug 2, 2018

so, stop whining/demanding, fix it, send PRs and I'll merge it...

@james-hr
Copy link

james-hr commented Aug 2, 2018

Hey, sorry, I did not mean to make it look like I am demanding or whining, was just reporting that it does not work for me. I know jack about Lua, so I would likely not be able to help at all in that regard.

I can, however, send some $ for coffee, if that will help expand this project.

I know wrt devs are fun to work with at times, so sorry that they are not playing along and your are having issues with them, hopefully that will be resolved soon.

Thank you for giving us an alternative to the otherwise god awful UI :-)

@chron0
Copy link
Member

chron0 commented Aug 4, 2018

@james-hr: kek, nothing to be sorry about, it's not like I was addressing you specifically :)

In any case, don't worry about the lua, the grunt work is in the css due to a lot of missing ids/classes, so we basically often have to iterate over elements/classes to get a match. This is stupid/tedious work, because as soon as something is changed/added upstream, the counts/elements won't match the css anymore.
I've darkmattered a couple of themeable things, including octoprint, and it's only openwrt/lede which makes it extremely hard for people to create and maintain consistent UX for luci, part of the reason why it looks like it does and why there are so few alternatives...

If I would have the time, I'd focus on completely rewriting it, drop jquery as a dep, to make it even smaller and work closely with whoever is behind luci, to get a better structure for classes and theme capabilities instead of dealing with inconsistent html pages and manually massaging DOM with jquery and wild css selectors.

@jow-
Copy link

jow- commented Aug 8, 2018

nah, I just don't prioritize the precious little time I have to push projects only to get into stupid arguments with openwrt dev crowd. I tried to get it merged but as you can check on the ML they didn't like that my email adress is chrono@foo so it got rejected.

Mind to elaborate on that? I neither see past pull requests at https://github.com/openwrt/luci/pulls?utf8=%E2%9C%93&q=darkmatter nor https://github.com/openwrt/openwrt/pulls?utf8=%E2%9C%93&q=darkmatter+ nor https://github.com/openwrt/packages/pulls?utf8=%E2%9C%93&q=darkmatter

The only mailing list conversation matching the keyword "darkmatter" I could find in the archive was https://www.mail-archive.com/[email protected]/msg40567.html and https://www.mail-archive.com/[email protected]/msg40570.html

At which point did "the openwrt dev crowd" complain about your e-mail address, let alone use it as a basis for rejecting a PR? I am genuinely interested because I am sure you never approached me or the LuCI project on github. The first time I learned about Darkmatter was when I got bug reports about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants