OpenConext / OpenConext-dashboard

Dashboard for IdP administrators to view and adminster SP connections
Apache License 2.0
8 stars 12 forks source link

Alignment edit/delete knoppen bij autorisatieregels #256

Closed phavekes closed 1 week ago

phavekes commented 1 week ago

This issue is imported from pivotal - Originaly created at Dec 14, 2016 by Arnout Terpstra

De alignment van de edit en delete knoppen valt buiten de container waar ze in zouden moeten zitten. Cre"eert lelijke scrollbars

Dit was op de "main" page ook zo, maar is wel gefixt.

phavekes commented 1 week ago

Ik kan dit niet reproduceren. Moeten we even contact over hebben. (Okke Harsta - Dec 27, 2016)

phavekes commented 1 week ago
@okkeh Als het scherm te klein is, waardoor niet alles binnen het window past, gebeurt dit. Zie screenshot (waar de edit/delete knoppen zelfs helemaal buiten het scherm vallen, heb niet eens horizontale scrollbars). (Arnout Terpstra - Dec 28, 2016)
phavekes commented 1 week ago

Ik heb de min-width verlaagd naar 1080. Dat maakt de app nog lang niet responsive maar het lost wel het probleem op bij kleinere beeldschermen (Okke Harsta - Dec 28, 2016)

phavekes commented 1 week ago

Het probleem is dat als de knoppen buiten beeld vallen, er geen horizontale scrollbar verschijnt. De enige manier om de knoppen te kunnen zien is dus het scherm vergroten (of uitzoomen). Dit geldt ook voor de connect knop op de main page. Het zou dus mooi zijn als het scherm te klein is, je in ieder geval nog naar rechts kunt scrollen om de knoppen te zien. (Arnout Terpstra - Dec 29, 2016)

phavekes commented 1 week ago

https://github.com/OpenConext/OpenConext-dashboard/blob/develop/dashboard/src/stylesheets/base/page.sass#L6 (Okke Harsta - Dec 29, 2016)

phavekes commented 1 week ago

Er is een discrepantie tussen de lokale css in dev modus en de prod modus. Zie bijgevoegde screenshots. Als de discrepantie is opgelost dan is ook dit issue opgelost. Wordt vervolgd... (Okke Harsta - Dec 29, 2016)

phavekes commented 1 week ago

(Okke Harsta - Dec 29, 2016)