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

feat: Nowy layout #196

Merged
merged 50 commits into from
Jan 31, 2023
Merged

feat: Nowy layout #196

merged 50 commits into from
Jan 31, 2023

Conversation

Wojtazzzz
Copy link
Contributor

@Wojtazzzz Wojtazzzz commented Jan 21, 2023

Zmiany wprowadzają odświeżony wygląd dla całego serwisu. Layout został zaproponowany w issue #140. Większość styli napisanych w CSS Modules zastała zastąpiona Tailwindem. W kilku miejscach logika z komponentów została wydzielona do hooków. Zostały również wprowadzone kosmetyczne zmiany w panelu administracyjnym.

BREAKING CHANGE: Nowy Layout

Resolves #140, #183, #111

api-helpers/articles.ts Outdated Show resolved Hide resolved
app/(articles)/[displayStyle]/[page]/page.tsx Show resolved Hide resolved
app/(content)/admin/[blogsType]/page.tsx Outdated Show resolved Hide resolved
app/(content)/admin/blogs/[blogId]/page.tsx Outdated Show resolved Hide resolved
app/(content)/artykuly/[slug]/page.tsx Outdated Show resolved Hide resolved
components/BlogsGrid/BlogsGrid.tsx Outdated Show resolved Hide resolved
styles/global.scss Outdated Show resolved Hide resolved
styles/global.scss Outdated Show resolved Hide resolved
utils/fetchBlogsForGrid.ts Show resolved Hide resolved
app/(articles)/[displayStyle]/[page]/page.tsx Outdated Show resolved Hide resolved
@typeofweb
Copy link
Member

typeofweb commented Jan 23, 2023

Przy wyszukiwaniu nie wyświetlają się favicony:
Screenshot 2023-01-23 at 14 38 07

Link czasem nachodzi na tytuł bloga:
Screenshot 2023-01-23 at 14 40 21

Ikonki nie są równo z tekstem:
Screenshot 2023-01-23 at 14 41 00
Screenshot 2023-01-23 at 14 42 52

Ten font wydaje się być za duży, spróbujmy go zmniejszyć:
Screenshot 2023-01-23 at 14 44 27

Między nagłówkiem, tekstem i wyszukiwarką praktycznie nia ma marginesów. Pod wyszukiwarką jest ogromny margines. Nie wygląda to najlepiej:
Screenshot 2023-01-23 at 14 45 08

Tytuł bloga ma za mały kontrast w stosunku do tła.
Przycisk z lewej strony powinien być wyrównany:
Screenshot 2023-01-23 at 14 46 52

Brak marginesu między CTA a przyciskiem. Ogromny margines nad CTA.
Brak odstępu między akapitami tekstu („Artykuł Priorytetyzacja i estymacja zadań pochodzi z serwisu…” to osobny akapit).
Screenshot 2023-01-23 at 14 47 22

Na mobile wyszukiwarka jest szersza niż reszta treści:
Screenshot 2023-01-23 at 14 49 23

components/ArticleTile/ArticleTile.tsx Outdated Show resolved Hide resolved
app/(content)/o-serwisie/page.tsx Outdated Show resolved Hide resolved
app/(content)/o-serwisie/page.tsx Outdated Show resolved Hide resolved
app/(content)/artykuly/[slug]/page.tsx Outdated Show resolved Hide resolved
components/BlogsGrid/BlogsGrid.tsx Outdated Show resolved Hide resolved
styles/tailwind.css Outdated Show resolved Hide resolved
tailwind.config.js Outdated Show resolved Hide resolved
@typeofweb
Copy link
Member

Czasem coś się tu rozjeżdża, ale nie wiem czemu:

F99197D4-87D5-4F27-BBD5-0B6F5CCD9122

FCF0676F-BCBB-4BB3-A960-EDB89E6A76DD

Ten zielony kolor trzeba zamienić na czerwony albo czerwony wszędzie na zielony:

C27CFED3-4617-48BB-BD83-9BD2AA5E2104

Tu jest bardzo dużo pustego miejsca:

F181BD8C-C10F-468E-B78C-8D435D47F0A0

Ikonki są malutkie, a kliknięcie w przycisk nie zamyka menu:

E80B9744-31BB-497F-A000-9E8C9968E504

Odstępy są nierówne:

AD92C86D-0632-43D9-A1BA-D634176A67C4

@typeofweb typeofweb merged commit e226b6f into develop Jan 31, 2023
@typeofweb typeofweb deleted the feat-Nowy-layout branch January 31, 2023 11:51
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.

[Front / Design] Redesign strony
3 participants