feat(sidebar): position over content

smart-village-solutions / smart-village-app-cms

A CMS based on JSON schema of smart-village api

GNU General Public License v3.0

7 stars 2 forks source link

feat(sidebar): position over content #66

Closed donni106 closed 3 years ago

donni106 commented 3 years ago

this places the sidebar over the page content
it always renders initially in toggled mode now to not cover page contents
the page itself needs to move the width of the toggled sidebar to the left, so that contents are not covered
therefore on mobile the top toggler needs to move as well if the sidebar is opened

SVA-206

please play around to see if it feels ok in several places

Screenshots

Bildschirmfoto 2021-06-10 um 17 50 40

theresastone commented 3 years ago

Sieht gut aus! Ging dein branch nicht von 206-CMS-Overfläche-Anpassungen ab? Dann wird es bestimmt merge Konflikte geben, weil ich bei 206 auch an der Sidebar gearbeitet habe (zum Beispiel sollte beim Login die Sidebar ganz verschwinden).

donni106 commented 3 years ago

Sieht gut aus! Ging dein branch nicht von 206-CMS-Overfläche-Anpassungen ab? Dann wird es bestimmt merge Konflikte geben, weil ich bei 206 auch an der Sidebar gearbeitet habe (zum Beispiel sollte beim Login die Sidebar ganz verschwinden).

danke! werde den PR für SVA-206 zuerst mergen und dann die neuere PRs daraufhin aufarbeiten.

codeclimate[bot] commented 3 years ago

Code Climate has analyzed commit 4ba32edb and detected 10 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Complexity	2
Style	8

View more on Code Climate.