Skip to content

add task solution#2747

Open
jkumorek01 wants to merge 6 commits into
mate-academy:masterfrom
jkumorek01:develop
Open

add task solution#2747
jkumorek01 wants to merge 6 commits into
mate-academy:masterfrom
jkumorek01:develop

Conversation

@jkumorek01

@jkumorek01 jkumorek01 commented May 4, 2026

Copy link
Copy Markdown

first task solution + demo link: https://jkumorek01.github.io/layout_miami/

@danon321 danon321 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

need DEMO link

@jkumorek01 jkumorek01 requested a review from danon321 May 4, 2026 17:48

@natalia-klonowska natalia-klonowska left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • imgs should maintain proper proportions so that they don’t look cropped or zoomed in
Image
  • icons shouldn't change position on hover
Image

CHECKLIST:

  • Add a favicon
  • Change text color on hover for phone, email and address
  • When you click on phone icon or phone number in contacts section, make sure that there is no 404 error, make it a real link to start a call on device
  • When clicking on any location / address - prevent errors and make it to open location in Google Maps
  • Pictures in Gallery should increase on hover
  • Placeholders in the forms suggest what to enter; apply validation of the form fields (required, email / tel etc.), then it is clear in what format to enter the data
  • Form shouldn't be submitted if some of the fields are not filled
  • Page shouldn't be reloaded on form submit
  • disable page scrolling under the menu

@danon321 danon321 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Desktop

Image

Tutaj na figmie jak zobaczysz wszystkie elementy maja te sama szerokosc. Mozesz stworzyc specjalna klase pod to container ktora bedzie zamykała twoj kontent w pudełku z odpowiednią

U ciebie kazdy z tych elementow zyje własnym zyciem:

Image

(na tym screenie na maksa oddaliłem ekran ale i tak elementy powinny trzymac sie dobrze)

@jkumorek01 jkumorek01 requested a review from danon321 May 12, 2026 20:58

@danon321 danon321 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Image

Nic sie nie zmieniło w linku do DEMO
https://jkumorek01.github.io/layout_miami/

@jkumorek01 jkumorek01 requested a review from danon321 May 14, 2026 10:47
@jkumorek01

Copy link
Copy Markdown
Author

I have updated the demo link

@natalia-klonowska natalia-klonowska left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mobile

  • pojawia ci się scroll poziomy. tytuł i ikony są zbyt duże oraz cała nawigacja jest zbyt przesunięta w dół
Image
  • wszystkie tytuły są zbyt duże
Image Image
  • niektóre zdjęcia są błędnie przybliżone. w drugim wierszu brakuje jednego zdjęcia. opisy pod zdjęciami powinny mieć większe odstępy miedzy wierszami. button powinien rozciągać się aż do miejsca gdzie kończą się pozostałe elementy sekcji
Image
  • paddingi sekcji są zbyt duże
Image

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.

3 participants