Sidebar for multiple course authors

[p15zerv]

Σχετικό Issue

closes site-gr/#121

Demo: https://adoring-yonath-300689.netlify.app/courses/semantic-and-social-web/

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

-- Αλλαγή του sidebar ώστε να εμφανίζονται πολλαπλοί authors σε κάθε μάθημα.

Σημειώσεις

1. Έκανα έναν έλεγχο μήπως χαλάει κάτι με τους πολλαπλούς authors, το μόνο που εντόπισα είναι στη σελίδα κάθε καθηγητή που αναγράφει τα μαθήματα του καθενός: για τα μαθήματα με πολλούς καθηγητές, αυτά δε συμπεριλαμβάνονται στη σελίδα τους. Θα μπορούσε ίσως να μπει ως ένα εύκολο issue στο site.
2. Στο site αυτή τη στιγμή κανένα μάθημα δεν έχει πολλαπλούς καθηγητές, οπότε η αλλαγή δεν εμφανίζεται κάπου. Θα μπορούσα να πειράξω ένα μάθημα (ίσως χωρίς PR για τόσο μικρή αλλαγή;) αν το θεωρεί κάποιος πρωτιμότερο.

[ghost]

Υπάρχουν αλλαγές στο ίδιο αρχειο sidebar.html με το #26 αλλα φαίνεται ότι δεν υπάρχουν conflicts..

[Spirosvw]

Φαίνεται πολύ καλό το παράδειγμα. Ως προς τις αλλαγές του sidebar.html σε σχέση με το #26 είναι σε διαφορετικά σημεία του αρχείου όπως ανέφερε και ο @constantinexisc. Δεν θεωρώ απαραίτητη την αλλαγή που προτείνεις στις σημειώσεις. Όπως ανέφερες θα ήταν καλύτερο να μπει σαν εύκολο issue για μελλοντικούς/νέους contributors.

[ghost]

Τέλεια, καθαρές οι αλλαγές και δουλεύει όπως πρέπει. Ίσως να ανοίξεις 2 issue στο αποθετήριο του site:

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

Και το δεύτερο για τα μαθήματα στην σελίδα των καθηγητών όπως είπες.

[p15zerv]

@Spirosvw αν δε σε πειράζει, αφού ήδη το κοίταξες, άφησε το και σε ένα review για να προχωράει!

[GIANNIS-AGGELIS]

@p15zerv ωραία δουλιά μου λειτουργεί κανονικά

[provopoulos]

Πολύ χρήσιμη λειτουργία και φαίνεται να ενσωματώνεται ομαλά με το θέμα του αποθετηρίου. Μπράβο!
Ωστόσο, ίσως θα ήταν καλό να ζητήσουμε και την γνώμη του @JimDragon καθώς οι αλλαγές έχουν γίνει σε βασικά αρχεία, απαραίτητα για την αναβαθμισιμότητα του θέματος.
https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/author-profile.html
https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/sidebar.html
https://mmistakes.github.io/minimal-mistakes/docs/upgrading/#update-files-manually

_includes | Replace all. Apply edits if you customized any includes.

[dimpram]

Πολύ χρήσιμη λειτουργία και φαίνεται να ενσωματώνεται ομαλά με το θέμα του αποθετηρίου. Μπράβο!
Ωστόσο, ίσως θα ήταν καλό να ζητήσουμε και την γνώμη του @JimDragon καθώς οι αλλαγές έχουν γίνει σε βασικά αρχεία, απαραίτητα για την αναβαθμισιμότητα του θέματος.

Ευχαριστώ για το tag και ναι όντως θα προκαλέσει conflict το οποίο επαναφέρει στην συζήτηση για άλλη μια φορά το πρόβλημα των conflicts. Υπάρχουν 2 σενάρια αντιμετόπισης αυτής της κατάστασης.

1. (Το οποίο νομίζω είναι αυτό που επικρατεί) Ο καθένας κάνει οτι αλλαγή θέλει και όταν έρθει η στιγμή για το δικό μου merge τα ξανακοιτάω όλα όσα έχουν conflicts (συνεπώς κάνοντας διπλή δουλειά)

2. Κάνουμε merge πρώτα το δικό μου και μετά ακολουθούν τα υπόλοιπα PRs αναπροσαρμοσμένα στο νέο θέμα.

Anyway πέρα από αυτό, @p15zerv νομίζω θα μπορούσες να προσθέσεις λίγο padding/margin αναμέσα στους καθηγητές για είναι πιο "τακτοποιημένο" αν θες

[p15zerv]

Πολύ χρήσιμη λειτουργία και φαίνεται να ενσωματώνεται ομαλά με το θέμα του αποθετηρίου. Μπράβο!
Ωστόσο, ίσως θα ήταν καλό να ζητήσουμε και την γνώμη του @JimDragon καθώς οι αλλαγές έχουν γίνει σε βασικά αρχεία, απαραίτητα για την αναβαθμισιμότητα του θέματος.
https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/author-profile.html
https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/sidebar.html
https://mmistakes.github.io/minimal-mistakes/docs/upgrading/#update-files-manually

_includes | Replace all. Apply edits if you customized any includes.

Δεκτό, αν και αυτό είναι γενικότερο θέμα για το #22, καθώς έχουν πλέον τροποποιηθεί αρκετά αρχεία στα includes, π.χ. #24 #26

Anyway πέρα από αυτό, @p15zerv νομίζω θα μπορούσες να προσθέσεις λίγο padding/margin αναμέσα στους καθηγητές για είναι πιο "τακτοποιημένο" αν θες

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