Portfolio Project#2750
Conversation
danon321
left a comment
danon321
left a comment
There was a problem hiding this comment.
Desktop (rozdzielczosc 1920x1080)
Jak widzisz elementy uciekają na lewą strone.
Mobile (rozdzielczosc 375 x 667)
Jak klikam ikonke do otwarcia menu mobilnego (menu burger icon) to ikonka zamkniecia tego menu powinna byc w tym samym dokładnie miejscu.
|
wprowadziłem poprawki |
|
wyskoczyły mi jeszcze rozjazdy w kilku miejscach ale je naprawiłem. |
natalia-klonowska
left a comment
natalia-klonowska
left a comment
There was a problem hiding this comment.
- marginesy boczne dla headera i menu powinny być inne w wersji tabletowej
- odstęp od krawędzi między logo a tytułem ma być dokładnie taki sam
- popraw stylowanie tytułu w headerze oraz linków w menu (font-size, font-weight, underline)
- zbyt duża przerwa między formularzem a sekcją contacts
- inputy są odrobinę zbyt wysokie
- dodaj stylowanie dla placeholdera message
- zwiększ odstęp między elementami
- popraw font-size i font-weight
między wersją tabletową a desktopową zdjęcie nie rozciąga się na całą szerokość oraz sekcja ze zdjęciami lepiej będzie wyglądać wycentrowana
|
poprawki wprowadzone., strona wyglada lepiej oraz wiem ze widok responsywny to nie okno przegladarki a tego nie wiedzialem 👍 Mam uwage do adresu mail na figmie sa dwa wiec wybralem ten nothing - jest na tablet i dekstop wiec ilosciowo wygral :) |
danon321
left a comment
danon321
left a comment
There was a problem hiding this comment.
Desktop
Tutaj na desktoppie troszeczke przesadziłes z odstępami.
Na figmie masz tak:
Tak samo Contact us. Tutaj również czcionka jest zła
Mobile
Tutaj odstępy się nie zgadzają
|
Zmiejszylem paddingi oraz poprawilem literowki. Co do czcionki w message to powtarzam ze na Figma jest taki sam font wiec aby okno message bylo focus - dalem lekko przyciemniony font. Ze wzgledu ze nie mam dokladnych wymiarow na Figmie to jest mniej wiecej czyli jak w description :) prosze o accept. |
zibi-dev
left a comment
zibi-dev
left a comment
There was a problem hiding this comment.
W wielu miejscach padding nie jest odpowiedni. Najlepiej, gdybyś ponownie przeszedł przez całość osobiście i upewnił się, że wszystko jest poprawne.
Postaraj się używać tylko jednego kierunku dla wertykalnego paddingu/marginesu, ponieważ w kilku miejscach wartości się sumują, przez co odległości są większe, niż powinny być. Później trudniej to utrzymać.
Sprawdź też z checklistą, czy wszystko, co jest tam wymienione, zostało zrobione. Widzę na przykład, że przy otwartej nawigacji nadal pojawia się scroll, a strona odświeża się po submitcie formularza.
natalia-klonowska
left a comment
natalia-klonowska
left a comment
There was a problem hiding this comment.
Co do czcionki w message to powtarzam ze na Figma jest taki sam font wiec aby okno message bylo focus - dalem lekko przyciemniony font. Ze wzgledu ze nie mam dokladnych wymiarow na Figmie to jest mniej wiecej czyli jak w description
jeśli font wygląda inaczej niż na figmie to zapewne oznacza że nie został zastosowany co możesz sprawdzić używając dev tools.
font który ustawiłeś na form został nadpisany przez domyślny styl elementu textarea. gdy dodasz font-family dla klasy form__input to widać że nadpisuje on ten domyślny i wygląda tak jak na figmie.
Ze wzgledu ze nie mam dokladnych wymiarow na Figmie to jest mniej wiecej czyli jak w description
jakie dokładne wymiary masz na myśli?
Dobrze nie zrozumielismy się - opierałem się na wyświetlaniu w jednej przeglądarce tj. Safari a tam było zgodnier z projektem. Dopiero na Chrome widziałem domyślny font. A wymiary dotyczyli tego ze nie da sie ich pobierac tak jak na filmach demo. Przynajmniej bez konta dev, wiec wymiary sa na oko pobierane. |
…rect forms submit
|
zmiany wprowadzone tak jak opis commit. Check w przegladarkach oraz check-lista - done. |
4 participants
Release 1.0 full description in DESCRIPTION-PROJECT file.
demo: https://h3depl.github.io/layout_miami/