search

Miiiluuu / eco-route-planner

a route planner with a focus on sustainability
0 stars 1 forks source link

layout header/footer #7

Closed Garvinus closed 1 year ago

Garvinus commented 1 year ago

Bitte einmal ansehen! ;-)

Miiiluuu commented 1 year ago

Dieser Branch wird noch weiter bearbeitet, noch nicht mergen, auf Rückmeldung von Kai warten :)

Garvinus commented 1 year ago

Kann gereviewed werden!

konstantinmuenster commented 1 year ago

Und es gibt noch einen Konflikt in der index.js Datei, den wir vor dem mergen auflösen müssen 😄 wenn du Hilfe brauchst, sag Bescheid 👍

Garvinus commented 1 year ago

Okay danke! Kümmer ich mich heute Nachmittag drum!Am 09.02.2023 03:07 schrieb Konstantin Münster @.***>:

@konstantinmuenster approved this pull request.

Top 🎉 nur zwei Kleinigkeiten, die man noch verbessern kann. Aber ansonsten 1A 💪

In pages/index.js:

export default function Home() {

return (

 <div>

Das kann wahrscheinlich gelöscht werden?

In components/Layout/Layout.js:

export function Layout(props) {

return (

 <div>

Eine kleine Optimierung wäre, dass ein Tag immer auch eine alt Property haben sollte. Das verbessert die Accessibility sofern z.B. Nutzer deine Webseite mit einem Screenreader aufrufen.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Garvinus commented 1 year ago

Resolved conflicts and merged.