Προσθήκη λειτουργίας theme switch button

KuhakuNeko commented 2 years ago

Δήλωση Θέματος

Σύντομη περιγραφή των επικείμενων αλλαγών: Θα υλοποιήσω ένα κουμπί το οποίο θα επιτρέπει στο χρήστη την εναλλαγή μεταξύ χρωματικών θεμάτων του site ώστε να μην περιορίζετε στο ήδη υπάρχον mint theme. Κάνοντας κλικ στο κουμπί θα γίνετε εναλλαγή από το default θέμα σε ένα άλλο. Η ιδέα είναι το δεύτερο θέμα να είναι σκοτεινό αλλά μπορώ αντί να δημιουργήσω ως δεύτερο θέμα ένα custom dark theme να μπεί απλά κάτι άλλο. Με την προσθήκη αυτής της λειτουργίας το site ξεφεύγει από την χρήση του έτοιμου, συν ότι η αλλαγή σχετίζεται με την εισαγωγή μου. Η υλοποίηση γίνετε με html, css/scss, js, και κώδικα liquid.
Εκτιμώμενη δυσκολία: hard
Σχετικοί συνεργάτες: @Orfeaslambrou @fireleaf22
Σχετικοί σύνδεσμοι: Δυναμική αλλαγή του background της κεντρικής σελίδας με βάση την συσκευή προβολής #396

DEMO: https://epic-hamilton-p2019108.netlify.app/

Υπενθυμίσεις

[X] Έχω βάλει σύντομο και μοναδικό τίτλο (π.χ., το όνομα της σελίδας ή της ανακοίνωσης)
[X] Έχω ενημερώσει τα παραπάνω πεδία με περισσότερες λεπτομέρειες, όπως σύνδεσμος στην σελίδα που αλλάζω ή/και σύνδεσμο στην αντίστοιχη σελίδα της επίσημης σελίδας με την ενημερωμένη πληροφορία.
[X] Έχω ελέγξει ότι το περιέχομενο των αλλαγών μου δεν το έχει δηλώσει κάποιος άλλος σε προηγούμενο issue ανοιχτό ή κλειστό
[X] Πριν προχωρήσω σε pull request θα πάρω ετικέτα green light
[X] Για την δεύτερη συνεισφορά έχω υλοποιήσει μια νέα λειτουργία και έχω εντοπίσει δύο σχετικούς αξιολογητές όπως φαίνεται στο λινκ που δίνω παραπάνω, διαφορετικά δεν γίνονται δεκτά θέματα που αφορούν απλή συνεισφορά περιεχομένου όπως στην πρώτη συνεισφορά

Tsimpliarakis commented 2 years ago

Πολύ καλύτερο απο το 1ο σου Issue, απλά αφού μπήκες που μπήκες στον κόπο, θα πρότεινα να βάλεις Dark mode.

Orfeaslambrou commented 2 years ago

Πολύ καλή προσπάθεια, δουλεύει άψογα.Το δεύτερο θέμα αρκεί έτσι πως έχει γίνει ,δεν χρειάζεται πιστεύω dark mode.Για μένα παίρνει οκ.

diogenisAl commented 2 years ago

@KuhakuNeko σε ποιο αρχείο/αποθετήριο θα γίνουν οι αλλαγές; Παράλληλα, περιμένουμε την απάντηση από τον @fireleaf22

KuhakuNeko commented 2 years ago

@diogenisAl Χρειάστηκε να γίνουν αλλαγές και στα δύο αποθετήρια αλλά κυρίως στο minimal mistakes. Τα αρχεία που άλλαξα ήταν αρκετά, όχι μόνο ένα, και χρειάστηκε να δημιουργήσω και μερικά δικά μου. Οπότε, όπως και την άλλη φορά, θα χρειαστούν 2 PR.

epidrome commented 2 years ago

τεχνικά είναι πράγματι μια δύσκολη αλλαγή που δείχνει την αντίστοιχη κατανόηση του θέματος και της σελίδας

αρχιτεκτονικά όμως είναι μια μάλλον κακή προσθήκη, καθώς μεταφέρει στην διεπαφή της ιστοσελίδας κώδικα που κανονικά τρέχει "στατικά" μια φορά στην παραγωγή της σελίδας

με άλλα λόγια φαίνεται να υπάρχει διπλασιασμός του κώδικα και ταυτόχρονα επιβάρυνση της διεπαφή χωρίς κάποιο σημαντικό όφελος, δλδ τα δύο αμαρτήματα κατά άλαν κέη ;)

το πιο ενδιαφέρον είναι πως επιβεβαιώνει τις αρχικές επιφυλάξεις του @KuhakuNeko για την δυσκολία εντοπισμού 2ου θέματος. προφανώς ο βαθμός θα είναι στην ζώνη του άριστου, και στην κορυφή αυτής της ζώνης αν ενημερωθεί η τεκμηρίωση ώστε τα νέα μέλη να μπορούν να αποφύγουν τυπικά εμπόδια όπως αυτό.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

ioniodi / sitegr