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

fixate the logout button to the right-side of the nav #20

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Julian-Marco
Copy link
Collaborator

No description provided.

@marcelruland marcelruland self-requested a review November 21, 2023 06:43
Copy link

@marcelruland marcelruland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mir erschliesst sich der Grund nicht ganz? Warum soll der Logout-Button nach rechts?

@@ -32,8 +32,8 @@
.logout-button {
margin-top: 30px;
margin-bottom: 30px;
padding-right: 16px;
margin-left: auto
margin-left: 240px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitte ohne hardcoded Pixel values. Mit ner flexbox geht das ohne, dass es zusammenbricht, wenn sich die Breite der Navbar aendert.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Habe leider keine elegante Möglichkeit gefunden gestern. Im Template steht der Button in dem selben mat-list-item wie der Username und das Logo, und die werden auch gleichmäßig verteilt, aber irgendwie wird der Platz in der Zeile nie richtig ausgenutzt und ist eher links-bündig. https://github.com/agri-gaia/DataDashboard/blob/main/src/modules/app/components/navigation/navigation.component.html#L19

@Julian-Marco
Copy link
Collaborator Author

Mir erschliesst sich der Grund nicht ganz? Warum soll der Logout-Button nach rechts?

Rein kosmetisch. Der klebt sonst immer so komisch am Usernamen und ändert seine Position je nach Länge des Namens und Größe des Logos. Außerdem ist es dann konsistent mit unserer Plattform-UI.

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

Successfully merging this pull request may close these issues.

2 participants