search

mworion / uzsu_widget

JAVA Script Widget for use in smarthome/ FHEM / smartvisu (with uzsu plugin)
6 stars 7 forks source link

Update visu.css #10

Closed bgewehr closed 9 years ago

bgewehr commented 9 years ago

Upper rule is unneccessary because standard width is set automatically. works fine and is less complex.

mworion commented 9 years ago

Merged manually

bgewehr commented 9 years ago

Hallo!

Der PR ist meiner Meinung nach nicht korrekt im Code angekommen, da es immer noch zwei Regeln in der CSS gibt, die den gleichen Filter benutzen (*='chrome'). Eine Regel soll für die NICHT Chrome Browser sein und ist eigentlich überflüssig (weil das Standardverhalten ohne Anpassung auskommt) und eine Regel (die untere) muss bleiben. Oder habe ich da was falsch verstanden?

mworion commented 9 years ago

Hallo Bernd,

ja sehe ich auch, obwohl in ihn gemerged hatte. Der ist auf erledigt. Da habe ich wohl etwas falsch gemacht (hatte ich noch nicht häufig gemacht auf Github). Wenn es Dir nichts ausmacht, bitte einen neuen stellen, den ich dann wieder direkt reinnehme. Ich will nachvollziehen, was ich da falsch gemacht habe. Danke Dir !

Michel

mworion commented 9 years ago

Hallo Bernd,

ja sehe ich auch, obwohl in ihn gemerged hatte. Der ist auf erledigt. Da habe ich wohl etwas falsch gemacht (hatte ich noch nicht häufig gemacht auf Github). Wenn es Dir nichts ausmacht, bitte einen neuen stellen, den ich dann wieder direkt reinnehme. Ich will nachvollziehen, was ich da falsch gemacht habe. Danke Dir !

Michel