Sidebar on "ects-calculator" and "courses" staying in your screen while scrolling

[alexpoulis]

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

• Σύντομη περιγραφή των επικείμενων αλλαγών:
  - Το sidebar στις σελίδες «ects calculator» και «courses» όπως κάνεις scroll down δεν «ερχόταν» και αυτό μαζί και έμενε στην κορυφή της σελίδας(demo πριν την αλλαγή για το ects-calculator, και demo πριν την αλλαγή για το courses).
  -Η αλλαγή που προτείνω είναι να βάλω τα sidebar menu να «κατεβαίνουν» όπως κάνεις scroll down και για την λύση στο πρόβλημα που αναφέρει ο συνεργάτης παρακάτω έβαλα και ένα scroll bar στο πινακάκι(demo μετά την αλλαγή για το ects-calculator, demo μετά την αλλαγή για το courses).
• Η αλλαγή που θα κάνω θα γίνει σε αυτό το repository και συγκεκριμένα θα αλλάξω στην πρώτη γραμμή το class από sidebar__right σε sidebar__right__with__scroll__bar sticky το οποίο είναι ένα νέο class το οποίο βρίσκεται εδώ και συγκεκριμένα στην γραμμή 93(αρχείο μετά την αλλαγή).
• Εκτιμώμενη δυσκολία: medium
• Σχετικοί συνεργατές: @p19kala @gordthrow
• Σχετικοί σύνδεσμοι (π.χ., στην επίσημη ιστοσελίδα, προηγούμενα σχετικά θέματα): Main menu staying on top while scrolling #382

Απορίες σχετικά με το θέμα (αν υπάρχουν)

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

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

[p19kala]

Καλη ιδεα, αλλα το προβλημα με αυτο ειναι οτι το sidebar ειναι οριζόντια στήλη και πολυ μεγαλη. Όσο κάνεις scroll, η μορφη του μενει η ιδια στην οθονη και ποτε δεν θα δεις τις επομενες προσθηκες, εκτος αν κανεις zoom out.

Παραδειγμα, ειμαι στο ECTS calculator. Εχω φτασει αρκετα κατω και ακομη δεν μπορω να δω περα απο το ΣΤ' εξαμηνο. Για να πατησω ενα συνδεσμο μετα απο το ΣΤ' εξαμηνο, πρεπει να κανω scroll κατω απο το Η' εξαμηνο (οπως δειχνω μετα) αλλα τοτε δεν εχει σημασια να πατησω κατι γιατι ειμαι ηδη εκει.

Mολις φτανω στο Η' εξαμηνο, αρχιζει να μην μενει πλεον πανω και να κατεβαινει. Αυτο μαλλον οφειλεται στο οτι το sidebar εχει φτασει στο τελος του border (του article? της κλασης page?):

Για το "courses", πρεπει να πας στο "Μεταπτυχιακο" για να αρχισει να προβαλει τα υπολοιπα αντικειμενα του sidebar.

Μια ιδεα που ειχα για αυτο ηταν να φτιαξω ενα drop-down menu για το sidebar ωστε αρχικά να δειχνει μονο "Προπτυχιακο", "Μεταπτυχιακο" και "Διδακτορικο" και οταν ο χρηστης κάνει click/hover να κανει expand με τα μαθηματα. Καθισα χθες 10 ωρες στο scss αρχείο του navigation και τελικα δεν το καταφερα. Αν μπορουσαμε να υλοποιησουμε αυτο, νομιζω θα λειτουργουσε και αυτο που προτεινεις. Αλλα ο χρονος πιεζει, θελει δεξιοτητες που δεν εχουμε και ο κωδικας σε καθε αρχειο ειναι ασχολιαστος...

[alexpoulis]

@p19kala Ευχαριστώ για την παρατήρηση, το είχα δει και εγώ αλλά δεν θεώρησα ότι ήταν τόσο σημαντικό. Η αλλαγή που δοκίμασα για την ώρα πρόχειρα είναι να αλλάξω το παρακάτω από 22px σε 17px το οποίο φαίνεται να λύνει το πρόβλημα και να εμφανίζει όλο το sidebar στην οθόνη. Τώρα επειδή δεν έχω χρόνο απαντάω με ένα screenshot με πρόχειρες αλλαγές που να δείχνουν την λύση και θα κάνω το demo από αύριο.

Και αν αυτό δεν δουλέψει νομίζω το ότι κάνοντας μία νέα class για αυτά τα sidebars των 2 σελίδων παρόμοια με αυτήν στις ανακοινώσεις με ένα slider στο πλάι θα ήταν μια ακόμα καλή λύση

[p19kala]

Για τη πρωτη προταση:
Αυτο μικραινει το font του text για ολο το text σε καθε σελιδα. Το bug συνεχιζει να υπαρχει και μαλιστα αυτη ειναι λυση μονο για το ects calculator, στο οποιο το sidebar τελειωνει στο Προπτυχιακο. Στις σπουδες, εχει και Μεταπτυχιακο και Διδακτορικο.
Επιπλεον, τωρα ειναι μικροτερο το sidebar και εμφανιζεται ολοκληρο σε συγκεκριμενη αναλυση οθονης. Εσυ εχεις μεγαλυτερη οθονη, οποτε σου εμφανιζεται καλα, αλλα αν βαλεις μια οθονη με μικροτερη αναλυση, το προβλημα θα συνεχιζει να υπαρχει. Π.χ. εγω εχω μικροτερη και το 14px ειναι που μου εμφανιζει ολοκληρο το sidebar στο ects calculator. Τωρα εχει γινει το text αρκετα μικρο.

Οσο για τη δευτερη προταση, ενα slider ακουγεται πραγματικα σαν λυση. Υπαρχει ηδη μια υποκλαση sidebar__right στο οποιο μπορεις να προσθεσεις αυτο το "nest selector" τελοσπαντων π.χ. "float". Οπως εχει με το sticky. Καλυτερα να τροποποιησεις την υποκλαση αντι για ολη τη κλαση sidebar.

[alexpoulis]

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

[p19kala]

σαν νέα class το σκεπτόμουνα σε περίπτωση που το class χρησιμοποιείται και κάπου αλλού οπότε ένα νέο class θα ήταν εκεί για να μην αλλάξει και τα υπόλοιπα που μπορεί να μην χρειάζονται την αλλαγή.

Εχεις δικιο. Ετσι, θα δουλευει μονο για το συγκεκριμενο sidebar και αν καποιος θελησει να εργαστει πανω στην sidebar__right, δεν θα υπαρξει προβλημα.

[alexpoulis]

Οκ θα το κάνω έτσι

[alexpoulis]

@p19kala Ανανεώθηκε το demo και νομίζω είναι το αποτέλεσμα το οποίο θέλαμε. Τώρα περιμένω τον συνεργάτη @gordthrow να δώσει και αυτός την γνώμη του και μετά περιμένω για greenlight για το pull request

[gordthrow]

Σαν ιδέα μου φαίνεται αρκετά καλή και σαν εκτέλεση από αυτά που είδα να έχεις συζητήσει με τον συμφοιτητή μας φαίνεται πως σε βοήθησαν στην κατανόηση του θέματος και φυσικά στην εκτέλεση του. Μετά από έλεγχο και του site από το κινητό φαίνεται οκ άρα για εμένα είσαι μια χαρά.

[diogenisAl]

@alexpoulis φτιάξε και το:

Τμήμα Πληροφορικήσ

στο demo σου

[alexpoulis]

Το έκανα δεν νομίζω να υπάρχει κάποιο άλλο θέμα

[epidrome]

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

[alexpoulis]

@epidrome
Η αλλαγή που κάνω είναι αλλαγή σημαντική διότι χωρίς αυτό που διορθώνω το πινακάκι στο πλάι, μετά από την μέση της ιστοσελίδας, δεν φαίνεται και αυτό είναι κάτι προβληματικό καθώς μέσα από αυτά που λέει αν δεν κάνω λάθος και ο Άλαν Κεη «Η πληροφορία που είναι δύσκολή να βρεθεί είναι σαν να μην υπάρχει” και στο πλαίσιο της ιστοσελίδας αυτής το να μην έχει το πινακάκι μετά από ένα μικρό scroll είναι σαν να μην το έχεις καθόλου(το να κάνεις scroll πάνω και κάτω για να βρείς αυτό που θέλεις δεν είναι βολικό και γλιτώνει πολύ χρόνο στον χρήστη). Τέλος θα αναφερθώ στο ότι το παραδοτέο ήταν πριν 22 μέρες κοντεύει μήνας τώρα μετά από ένα τόσο μεγάλο χρονικό διάστημα το να έρθετε και να ακυρώνεται ένα θέμα το οποίο να αναφέρω το ότι έχει δουλευτεί(αλλαγές σχετικά με το minimal-mistakes) μέσα στις γιορτές τώρα δηλαδή είναι κορόιδεμα. Το να απαιτείται να είμαστε εμείς στην ώρα μας και να υπάρχει τέτοια καθυστέρηση από την μεριά σας είναι δηλαδή τι να πω, αλλά εγώ θα το πιέσω για αυτό το θέμα γιατί όντος πιστεύω το ότι η αλλαγή είναι σημαντική όπως είπα και παραπάνω αλλά και σαν δυσκολία δεν είναι αλλαγή μιας γραμμής.

[epidrome]

1. στις 20 μέρες που αναφέρεις υπάρχουν και 17 μέρες γιορτών-αργίας. γενικά απαντάμε μέσα σε μέρες, και αν ερχόσουν στις συναντήσεις την εβδομάδα πριν/μετά τις γιορτές, άμεσα στο ντέμο σου.

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

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

Για την βαθμολόγηση του 2ου+ παραδοτέου αξιολογούμε βαθμό δυσκολίας και κυρίως την συμβατότητα με το πνεύμα του έργου. Είναι σχετικά εύκολο να γίνουν δύσκολες τεχνικές προσθήκες, οι οποίες όμως δεν ταιριάζουν με το πνεύμα του έργου και τελικά δεν γίνονται δεκτές. Για παράδειγμα, οι περισσότερες προσθήκες θα πρέπει να γίνουν στο αρχικό θέμα και όχι στο αντίγραφο του που διατηρούμε στο minimal-ionio, οπότε αυτές οι αλλαγές δεν είναι συμβατές και δεν γίνονται δεκτές ακόμη και αν είναι άρτιες τεχνικά.

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

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