Δυναμική αλλαγή background (2019027 / pull2)

p19zing commented 2 years ago

Σχετικό Issue

closes ioniodi/sitegr#396

Link προς το demo των αλλαγών Πρέπει να φορτωθεί από κινητό !

Σχετικό Pull Request στο minimal-ionio

Συνεργάτες

@Vuxxs , @p19vrou

Προτεινόμενες Αλλαγές

Όπως αναφέρεται στο Video background on home page, στην τελευταία παράγραφο (Εναλλακτικές Προσεγγίσεις), υλοποίησα την πρόταση για δυναμική αλλαγή του background της κεντρικής σελίδας με βάση την συσκευή, δηλαδή background video όταν ο user agent είναι PC/Laptop και εμφάνιση jpeg image όταν ο user agent είναι Smartphone/Tablet.

Η υλοποίηση έγινε με Javascript στο sitegr και κατά την φόρτωση της σελίδας, αν ο user agent είναι κινητό, σβήνει το iframe και θα το αντικαθιστά με εικόνα

Πρόσθεσα στο index.html το αρχείο της javascript (assets/js/index.js) και πρόσθεσα κατάλληλο css styling στο minimal-ionio για να φαίνεται σωστά η εικόνα.

..

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

[x] Έχω ανοίξει από πριν issue για τον καλό συντονισμό του project, το οποίο έχει πάρει το πράσινο φως με την αντίστοιχη ετικέτα
[x] Έχω ενημερώσει το issueNo παραπάνω με τον αριθμό του αντίστοιχου θέματος, ώστε να κλείσει αυτόματα με την αποδοχή αυτού του αιτήματος
[x] Έχω δημιουργήσει branch για τις αλλαγές

epidrome commented 2 years ago

γιατί στέλνεις εδώ αντί για το θέμα το αρχείο js?

p19zing commented 2 years ago

γιατί στέλνεις εδώ αντί για το θέμα το αρχείο js?

@epidrome Μετέφερα το index.js στο minimal-ionio

p19vrou commented 2 years ago

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

epidrome commented 2 years ago

μήπως και η αναφορά στο index.js πρέπει να πάει κάπου στο θέμα σε κάποιο layout-include?

ioniodi / sitegr