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

Avatar nachodzi na inny avatar na liście wiadomości #720

Open
danon opened this issue Feb 8, 2023 · 6 comments
Open

Avatar nachodzi na inny avatar na liście wiadomości #720

danon opened this issue Feb 8, 2023 · 6 comments

Comments

@danon
Copy link
Collaborator

danon commented Feb 8, 2023

image

@goldipl
Copy link
Contributor

goldipl commented Feb 11, 2023

Poprawka w CSS

.mw-100 {
    max-width: 100%!important;
    width: 35px;
    height: 35px;
    object-fit: contain;
}

@danon
Copy link
Collaborator Author

danon commented Feb 11, 2023

@goldipl Jak ustawiasz width i height, to max-width jest niepotrzebne. I swoją drogą żeby util mw-100 ustawiał stałą szerokośc też jest średnie, więc ja proponuje:

.avatar-size {
    object-fit: contain;
    width: 35px;
    height: 35px;
}

Druga sprawa jest taka, że raczej nie możemy tak po prostu ustalić stałych rozmiarów, bo ten sam util jest używany zarówno w powiadomieniach jak i na liście wiadomości. W powiadomieniach jest 35px/35px, w wiadomościach rozmiar to 45px/45px. Więc albo trzeba zrobić dwa osobne utilsy na powiadomienia i wiadomości; albo jakoś inteligentniej to zaprogramować.

@danon
Copy link
Collaborator Author

danon commented Feb 11, 2023

@wojtekxtx Tylko że ten styl mw-100 jest z bootstrapa, i tam ten !important jest niestety konieczny. W kodzie coyote'a raczej ich unikamy.

@adam-boduch
Copy link
Collaborator

@wojtekxtx @danon ma racje, to jest styl bootstrapa i nie powinien byc zmieniany.

@goldipl
Copy link
Contributor

goldipl commented Feb 12, 2023

@goldipl

  • pomiędzy wartością a !important spacja.
  • gdzieś czytałem, że !important jest niepolecane.

A tak btw, to co proponuje @danon brzmi lepiej

spacja może być, ale nie musi :) i pełna zgoda z tym co powyżej było pisane

@danon
Copy link
Collaborator Author

danon commented Feb 12, 2023

spacja może być, ale nie musi :) i pełna zgoda z tym co powyżej było pisane

Tej spacji pewnie nie ma w kodzie źródłowym, tylko bootstrap minifikuje .css, więc usuwa tą spacje, i potem ją widzisz w inspektorze. Także nie ma tematu.

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

No branches or pull requests

3 participants