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

Over page #295

Merged
merged 21 commits into from
Jan 14, 2025
Merged

Over page #295

merged 21 commits into from
Jan 14, 2025

Conversation

Nazneen05x
Copy link
Collaborator

@Nazneen05x Nazneen05x commented Dec 27, 2024

What does this change?

Ik heb een Over pagina gebouwd voor de Bieb in Bloei website, op deze pagina kun je achtergrondinformatie lezen over Bieb in Bloei. Deze pagina is gebouwd met Svelte (Sveltekit) en Hygraph. Ik heb het design van de Geveltuin pagina aangehouden, zodat er consistentie is op de website.

Ik verdeel mijn taak altijd in kleinere taken, daarom heb ik veel issues.
Resolves issue: #224 #244 #225 #226 #227 #228

Livesite

How Has This Been Tested?

Bekijk hier mijn testresultaten: testen testen testen
Bekijk hier de user test: testplan

  • User test
  • Accessibility test
  • Performance test
  • Device test (responsive)
  • Browser test

Images

over-page

How to review

  • Code checken (code conventions controleren)
  • Responsief checken
  • Attention to details

@Nazneen05x Nazneen05x requested a review from Annevd December 27, 2024 23:58
Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for over-pagina ready!

Name Link
🔨 Latest commit 054b3db
🔍 Latest deploy log https://app.netlify.com/sites/over-pagina/deploys/67801eb73ef931000861de6f
😎 Deploy Preview https://deploy-preview-295--over-pagina.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for biebinbloei ready!

Name Link
🔨 Latest commit 054b3db
🔍 Latest deploy log https://app.netlify.com/sites/biebinbloei/deploys/67801eb73735200008fc668e
😎 Deploy Preview https://deploy-preview-295--biebinbloei.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@Annevd Annevd left a comment

Choose a reason for hiding this comment

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

Ik heb wat feedback achtergelaten in comments, verder ziet de code er goed uit! :)

src/lib/index.js Show resolved Hide resolved
src/routes/+layout.svelte Show resolved Hide resolved
@Annevd
Copy link

Annevd commented Jan 7, 2025

De pagina is goed responsive. De enige opmerking die ik heb is het Bieb in Bloei logo dat opeens verschijnt op mobiel. Deze is best groot en zorgt voor veel lege ruimte ernaast. Ik vroeg me af wat de toevoeging van het logo hier is? En waarom alleen op mobiel en niet desktop?

Er staat al "Over Bieb in bloei' dus wellicht zou je het embleem ook weg kunnen laten.

@Nazneen05x
Copy link
Collaborator Author

Nazneen05x commented Jan 9, 2025

De pagina is goed responsive. De enige opmerking die ik heb is het Bieb in Bloei logo dat opeens verschijnt op mobiel. Deze is best groot en zorgt voor veel lege ruimte ernaast. Ik vroeg me af wat de toevoeging van het logo hier is? En waarom alleen op mobiel en niet desktop?

Er staat al "Over Bieb in bloei' dus wellicht zou je het embleem ook weg kunnen laten.

In de content staat een stukje tekst over het logo, daarom denk ik dat Mariska het logo heeft meegestuurd met de content.

De lege witruimte op de desktop was veel groter, zag er niet uit, daarom heb ik het weggelaten op de desktop. Op mobiel vind ik het meevallen. Ik ga proberen om het logo klein onderaan de tekst (content) te plaatsen.

Scherm­afbeelding 2025-01-09 om 16 10 45

@Annevd
Copy link

Annevd commented Jan 9, 2025

De pagina is goed responsive. De enige opmerking die ik heb is het Bieb in Bloei logo dat opeens verschijnt op mobiel. Deze is best groot en zorgt voor veel lege ruimte ernaast. Ik vroeg me af wat de toevoeging van het logo hier is? En waarom alleen op mobiel en niet desktop?
Er staat al "Over Bieb in bloei' dus wellicht zou je het embleem ook weg kunnen laten.

In de content staat een stukje tekst over het logo, daarom denk ik dat Mariska het logo heeft meegestuurd met de content.

De lege witruimte op de desktop was veel groter, zag er niet uit, daarom heb ik het weggelaten op de desktop. Op mobiel vind ik het meevallen. Ik ga proberen om het logo klein onderaan de tekst (content) te plaatsen.

Scherm­afbeelding 2025-01-09 om 16 10 45

Lijkt me een goed idee! Ik ben benieuwd. Anders zo laten, het is geen must have!

Ik heb de naam van een icon veranderd naar een betere naam voor mijn mede devs.
@Nazneen05x
Copy link
Collaborator Author

Nazneen05x commented Jan 9, 2025

De pagina is goed responsive. De enige opmerking die ik heb is het Bieb in Bloei logo dat opeens verschijnt op mobiel. Deze is best groot en zorgt voor veel lege ruimte ernaast. Ik vroeg me af wat de toevoeging van het logo hier is? En waarom alleen op mobiel en niet desktop?
Er staat al "Over Bieb in bloei' dus wellicht zou je het embleem ook weg kunnen laten.

In de content staat een stukje tekst over het logo, daarom denk ik dat Mariska het logo heeft meegestuurd met de content.
De lege witruimte op de desktop was veel groter, zag er niet uit, daarom heb ik het weggelaten op de desktop. Op mobiel vind ik het meevallen. Ik ga proberen om het logo klein onderaan de tekst (content) te plaatsen.
Scherm­afbeelding 2025-01-09 om 16 10 45

Lijkt me een goed idee! Ik ben benieuwd. Anders zo laten, het is geen must have!

Ik heb een isseu aangemaakt.

#304

@Nazneen05x Nazneen05x merged commit 78eb3f6 into release-candidate-bieb-in-bloei Jan 14, 2025
8 checks passed
@Nazneen05x Nazneen05x deleted the Nazneen-over-page branch January 14, 2025 13:30
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