Twitter Feed module about site-gr HOT 14 CLOSED

@epidrome

Βρηκα τι παει λαθος, αν ο browser που χρησιμοποιει καποιος κανει tracking block ( το facebook) τοτε δεν δουλευει το widget. Εκανα whitelist την σελιδα απο τις επιλογες στο address bar και δουλεψε.

'Ισως να χρειαστει και απο αλλους browsers οπως firefox/chrome/edge να τσεκαρει την συμπεριφορα τους ο @provopoulos .

from site-gr.

Ίσως με κάτι τέτοιο: https://www.detectadblock.com/
Αλλά δεν την θεωρώ κομψή λύση για τη θεματολογία της σελίδας.
Από την άλλη τα λεγόμενα ad-blockers (+trackers) δεν είναι η προεπιλογή στους δημοφιλείς browsers. Εμένα μου παρουσιάζεται το πρόβλημα στο Brave + Safari (iOS) επειδή το Brave έχει ενσωματωμένο ad-blocker και στο Safari επειδή έχω προσθέσει εφαρμογή φραγής περιεχομένου.

from site-gr.

@provopoulos
αν θες να στείλεις το αίτημα ώστε μετά να προχωρήσεις στο επόμενο παραδοτέο

στο τέλος της σελίδας έχει ένα παράδειγμα καλής χρήσης
https://www.pwc.com/gr/el/academy.html

from site-gr.

Demo: https://vibrant-albattani-753ac0.netlify.app/
Τελικά η αναπαράσταση των ανακοινώσεων, εργαστηρίων και projects έχει πολύ συγκεκριμένη και αυστηρή δομή καθορισμένη από την λειτουργία της feature_row() η οποία δεν παρέχει εγγενή υποστήριξη για δυναμικό περιεχόμενο τύπου Twitter Embedded Timeline.
Ο κώδικας ενσωμάτωσης του χρονολογίου χρειάστηκε τροποποιήσεις ώστε να ταιριάζει με την δομή απεικόνισης αντικειμένων της feature_row().

@Spirosvw

<div class="feature__wrapper">
...
...
...
<div>

^ Η ρητή κλήση της κλάσης feature__wrapper στο index.html αρχείο δεν είναι απαραίτητη καθώς καλείται αυτόματα ούτως ή άλλως μέσω της feature_row(). Με την ρητή κλήση δημιουργούνται 2 στιγμιότυπα της feature__wrapper για κάθε ένα feature_row που δημιουργούμε με αποτέλεσμα να χαλάει η διάταξη. Έπρεπε να αφαιρέσω την ρητή δήλωσή της από το index ώστε να μην σπάει η ενσωμάτωση του Twitter. Σχετικό C: provopoulos@56b7432
Τέλος, εφόσον χρειάστηκαν αλλαγές και στα 2 αποθετήρια θεώρησα καλή πρακτική να μεταφέρω το dynamic.html στο αποθετήριο του θέματος ώστε να βρίσκεται μαζί με τα υπόλοιπα: provopoulos/minimal-ionio@f734d83

@JimDragon
Ο Δημήτρης παρέθεσε μια εύλογη παρατήρηση. Ο μόνος τρόπος μέσω της αρχικής σελίδας για ανακατεύθυνση στα projects είναι εντός του feature row σε αντίθεση με τα εργαστήρια στα οποία παρέχεται πρόσβαση και από την μπάρα της επικεφαλίδας, επομένως θεώρησα καλύτερη την αντικατάσταση των εργαστηρίων εντός του feature row με το Twitter. Σε ευχαριστώ!

Με την σύμφωνη γνώμη όλων @epidrome, @andreanastil, @JimDragon & @Spirosvw, είμαι έτοιμος να προχωρήσω στα 2 PR με τις αλλαγές!

from site-gr.

Προσθήκη των ενημερώσεων της σελίδας του τμήματος στο Twitter στην αρχική σελίδα.

from site-gr.

Ενδεικτικό DEMO: https://gallant-saha-59cc7f.netlify.com/

from site-gr.

το ντέμο δεν φαίνεται να δουλεύει

from site-gr.

@constantinexisc
Το πρόβλημα του κ. Χωριανόπουλου ήταν ότι δεν λειτουργούσε το αρχικό DEMO διότι έκανα κάποιες αλλαγές και το ενημέρωσα κατά τη διάρκεια της τηλεδιάσκεψης.

Ωστόσο, το πρόβλημα που αναφέρεις το αντιμετώπισα κι εγώ και το ανέφερα στη συζήτηση που είχαμε. Παράξενο πάντως, χρησιμοποιώ 2-3 επεκτάσεις για την φραγή διαφημίσεων (+trackers) στο Firefox και τέτοιο πρόβλημα δεν έχω, εμφανίζεται το χρονολόγιο κανονικά. Υποθέτω πως διαφέρουν οι λίστες φραγών από browser σε browser καθώς το πρόβλημα μου παρουσιάστηκε στο Brave (Chromium-based).

Good catch, thanks @constantinexisc !

from site-gr.

μήπως υπάρχει τρόπος να ελέγχουμε την ύπαρξη φραγής;

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

from site-gr.

@provopoulos
επειδή με το adblock check ίσως το θέμα αυτό γίνεται κάπως δύσκολο, μερικές ιδέες για το επόμενο βήμα να πάρεις κάτι άλλο για εύκολο και:

1. να κρατήσεις αυτό για δύσκολο
2. να επιλέξεις άλλο δύσκολο ή ακόμη καλύτερα να ορίσεις ένα δύσκολο της επιλογής σου

from site-gr.

@epidrome
Πολύ χρήσιμη υλοποίηση, θα γράψω μερικά σχόλια για πιθανή μελλοντική αναδρομή και μόλις είμαι έτοιμος θα στείλω PR. Ευχαριστώ!

from site-gr.

@provopoulos πότε θα στείλεις???

γιατί φαίνεται πως ακόμη δεν έχεις στείλει το εύκολο για την προθεσμία 30 Μαρτίου
https://github.com/ioniodi/site-gr/pulls/provopoulos

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

from site-gr.

To module φαίνεται αρκετά χρήσιμο και λειτουργεί αρκετά καλά. Ίσως όμως να χρειαστούν αλλαγές στην τοποθέτηση του module. Στον δικό μου browser εμφανίζεται κάτω δεξιά και το Facebook πάνω στο κέντρο σε πολύ μεγαλύτερη έκταση. Μήπως στο μέλλον και τα δύο modules να καλύπτουν την ίδια έκταση απλά για λόγους αισθητικής.

from site-gr.

@epidrome
Ευχαριστώ για την υπενθύμιση. Ελπίζω μέχρι και το πέρας του ΣΚ να έχω στείλει το PR.

@Spirosvw
Ευχαριστώ πολύ για τα σχόλιά σου. Το DEMO παραπάνω είναι παλιό σε σχέση με αυτά που συζητήσαμε στο μάθημα. Το Facebook Embed παρουσιάζει κάποια προβλήματα - περιορισμούς (θα αναφερθώ σύντομα) επομένως αποφασίσαμε να το αντικαταστήσουμε με το Twitter module.
Αυτό που αποφασίσαμε να κάνουμε, είναι να το τοποθετήσουμε στην σειρά με τις στήλες 'Posts - Projects - Groups' αντικαθιστώντας μια εξ' αυτών (π.χ. Projects).

Θα ήθελα να σε ρωτήσω, σύμφωνα με το πρόσφατο PR που δημιούργησες: Δυναμική αλλαγή του περιεχομένου στην αρχική σελίδα: #94, αν θέλω να αντικαταστήσω τα Projects με το Twitter module, οι αλλαγές μου πρέπει να γίνουν στο index.html ή κανονικά στο εξωτερικό αποθετήριο του θέματος;

{% include dynamic.html id="projects" category="Projects" index=random %}
{% assign random = site.time | date: "%s%N" | modulo: site.groups.size %}
{% include feature_row %}

Ευχαριστώ πολύ!

from site-gr.