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

Filter Header Layout fixen #160

Closed
hupf opened this issue Apr 22, 2020 · 8 comments
Closed

Filter Header Layout fixen #160

hupf opened this issue Apr 22, 2020 · 8 comments
Assignees
Labels
bug Something isn't working module-AbsenzenAuswerten Feature fürs Modul "Absenzen auswerten" module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten"

Comments

@hupf
Copy link
Collaborator

hupf commented Apr 22, 2020

Das Layout des Filter-Headers funktioniert seit den Updates im IE11 nicht mehr:

image

Mit Commit 8fb4a45 von Sandro wurden aussderm CSS Statements eingeführt, welche keinen Sinn ergeben (floats, display:block mit flex Rules die gar nicht greifen). Fixen und allgemeines Aufräumen dieses Layouts.

@hupf hupf added bug Something isn't working module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten" module-AbsenzenAuswerten Feature fürs Modul "Absenzen auswerten" labels Apr 22, 2020
@hupf hupf changed the title Filter Header fixen Filter Header Layout fixen Apr 22, 2020
@schefbi
Copy link
Contributor

schefbi commented Apr 22, 2020

Auf dem aktuellen Build 10.12.2019 den wir in EventoWeb in Betrieb haben. Ist das Layout im IE11 einwandfrei.

image

Ich könnte mir aber vorstellen, dass es mit dem Upgrade der dependency ein Zusammenhang hat.

@hupf
Copy link
Collaborator Author

hupf commented Apr 22, 2020

@schefbi Das stimmt sogar. Ich würde trotzdem das Layout dort überarbeiten, es scheint mir etwas fragil implementiert zu sein.

@hupf hupf self-assigned this Apr 22, 2020
hupf added a commit that referenced this issue Apr 22, 2020
@hupf hupf assigned caebr and unassigned hupf Apr 22, 2020
@hupf hupf assigned hupf and unassigned caebr Apr 22, 2020
hupf added a commit that referenced this issue Apr 27, 2020
@hupf hupf assigned caebr and unassigned hupf Apr 27, 2020
hupf added a commit that referenced this issue Apr 28, 2020
@hupf hupf assigned schefbi and fbufbi and unassigned caebr Apr 28, 2020
@schefbi
Copy link
Contributor

schefbi commented Apr 30, 2020

@hupf Button Anzeigen margin-top: 2.7rem anstelle von 2rem

image

@schefbi schefbi assigned hupf and unassigned schefbi and fbufbi Apr 30, 2020
@hupf
Copy link
Collaborator Author

hupf commented Apr 30, 2020

@schefbi Was habt ihr für eine Base Font Size auf eurem Body?

@schefbi
Copy link
Contributor

schefbi commented May 4, 2020

@hupf 1.5rem ist die size auf dem body.

Heute Version Dezember 2019 sind buttons class so implementiert.
grafik

hupf added a commit that referenced this issue May 5, 2020
@hupf
Copy link
Collaborator Author

hupf commented May 5, 2020

Habe nun den margin-top wieder auf 31px gesetzt anstatt den Abstand zu berechnen. Eigentlich müssten wir die $font-base-size von Bootstrap auf 1.5rem setzen, wenn ihr das in der Produktion so habt. Die Frage ist, habt ihr dann bei euch auf dem html Tag die Default Browser Font Size auf 11px oder 12px überschrieben, damit es am Ende wieder gleich gross aussieht?

@hupf hupf assigned schefbi and unassigned hupf May 5, 2020
@schefbi
Copy link
Contributor

schefbi commented May 5, 2020

Auf dem html Tag ist die Size 11px. Test OK

@hupf
Copy link
Collaborator Author

hupf commented May 6, 2020

Habe nun folgenes geändert (in Commit 1ce3c59):

  • font-size: 11px auf html
  • font-size: 1.5rem auf body
  • $font-size-base: 1.5rem in den Bootstrap Variablen
  • Diverse Anpassungen der Abstände/Grössen, da diese nun nicht mehr stimmten

Somit sollten wir die gleiche Umgebung haben wie in der Produktion, kannst du dies mit dem aktuellsten Stand bei euch testen @schefbi ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working module-AbsenzenAuswerten Feature fürs Modul "Absenzen auswerten" module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten"
Projects
None yet
Development

No branches or pull requests

4 participants