Uprading to minimal mistakes 4.19.1 💎 #22
Conversation
Close #2092
* Replace jQuery Smooth Scroll with Smooth Scroll + Gumshoe * Change smooth scrolling speed * Set maximum smooth scroll duration * Remove Font Awesome CSS Ppseudo elements attribute from JS script Close #2050, close #2075
```yml search_placeholder_text : "Nhập từ khóa cần tìm..." results_found : "Kết quả tìm được" back_to_top : "Về đầu trang" ```
I just fixed some typos PT-BR, included translations for words that were not translated and changed some words to better describe the tag's meaning. This is how I use it in my Portuguese blog.
Due to unknown reason, Hindi translation of ui-text was missing (mmistakes/minimal-mistakes#2101) I have added it back from a previous pull request (mmistakes/minimal-mistakes#1888)
Due to unknown reason, Hindi translation of ui-text was missing (mmistakes/minimal-mistakes#2101) I have added it back from a previous pull request (mmistakes/minimal-mistakes#1888)
Allow adding JavaScript files after those bundled in the theme Close #2110
* 'master' of https://github.com/mmistakes/minimal-mistakes: Allow adding JavaScript files after the theme's (#2116)
* Update search_form.html Updates Issue #2122 by adding the correct input type and aria label. * update search_form.html should be area-placeholder. My mistake. Updates Issue #2122 * fix aria-placeholder Updates issue #2123 with a typo found by @mmistakes for liquid syntax. Close #2122
Ναι θα το δουλέψω αν και είχα σκοπό να τα γράψω μια και καλή στο wiki γιατι είναι πολλά. |
|
@JimDragon παρατήρησα στο live demo ότι όταν σκρολάρεις προς τα επάνω κυρίως, οπτικά φαίνεται σαν να επιλέγονται κάποια μαθήματα που περιέχονται στο πίνακα (με την έννοια ότι γίνεται μπλε το κουτάκι που περιέχει τον τύπο του μαθήματος π.χ. "Μαθήματα Επιλογής"). Δεν ξέρω αν το έχεις δει και εσύ και το έχεις διορθώσει, χωρίς να έχει γίνει update το demo. |
Δυσκολευομαι να καταλάβω ακριβώς τι εννοείς αλλά μου φαίνεται οτι είναι feature του upgraded theme. Αυτό που βλέπω εγώ είναι οτι ανάλογα σε ποιο σημείο είσαι στο page, αντίστοιχα σου κάνει μπλε το ονόνομα του segment στον πίνακα. |
Upgrade notes
Το config και το masthead που είναι με bold είναι τα αρχεία που θέλουν ιδιαίτερη προσοχή καθώς το config είναι πολύ σημαντικό για το jekyll και το masthead μου προκάλεσε errors γιατί δεν έκλειναν κάποια for loops. ΣυμπέρασμαΕίναι μια διαδικασία που θέλει προσοχή καθώς ορισμένες φορές είναι στην κρίση σου ποια αρχεία θα πρέπει να αλλάξουν/διαγραφούν και ποια όχι. Για να μπορείς να κρίνεις σωστά κατά την άποψή μου θα πρέπει πρώτων να γνωρίζεις καλά το file structure που ακολουθεί το jekyll και τις ιδιότητες του κάθε φακέλου/αρχείου καθώς και ποια αρχεία είναι δικά μας και ποια του minimal mistakes. Αυτά είναι τα πρόχειρα notes μου ελπίζω να σε κάλυψαν για αρχή. @constantinexisc |
There was a problem hiding this comment.
Αναφορικά με τον πίνακα μαθημάτων, δεν είχα καταλάβει ότι είναι feature του upgraded theme, αν και πάλι δεν το θεωρώ πολύ εύχρηστο. Σε κάθε άλλη περίπτωση, φαίνεται να είναι λειτουργικό το σύνολο του theme, ωστόσο πριν προχωρήσουμε σε merge, θα έλεγα να το κοιτάξουν και οι υπόλοιποι.
|
Αν σας είναι εύκολο @GIANNIS-AGGELIS @epidrome ρίξτε μια ματία γιατί γίνονται αλλαγές στο θέμα και θα πρέπει να το ξαναπροσαρμόζω συνεχώς. |
|
επειδή είναι σημαντική αλλαγή καλό είναι να έχουμε 2-3 αξιολογήσεις-τεστ πρώτα |
|
@AsteriosP @p15zerv μπορείτε να το αξιολογήσετε με προτεραιότητα ώστε να ενσωματωθεί πριν έρθουν νέα αιτήματα που θα δημιουργήσουν σύγκρουση; |
There was a problem hiding this comment.
Έχω ξεκινήσει να κοιτάζω μέσα στα αρχεία, αλλά θα χρειαστώ μερικές μέρες καθώς υπάρχουν αρκετές αλλαγές.
Στο μεταξύ κάποια πρώτα σχόλια, μετά από μια πρώτη περασιά:
- Αρκετά αρχεία έχουν να κάνουν με το github και δε μας ενδιαφέρουν τόσο. Π.χ. όλα τα αρχεία στο φάκελο
.githubκαι τοLICENSE.
- Αυτό δε συμπεριλαμβάνει τις αλλαγές στο .gitignore, οι οποίες φαίνονται χρήσιμες.
- Διάφορες αλλαγές έχουν να κάνουν με παραδοτέα άλλων και δεν ξέρω κατά πόσο τους επηρεάζουν χωρίς να δω αναλυτικά όλες τις αλλαγές. Π.χ. υπάρχουν αλλαγές σε search engines, tags στα posts, κάποιες αλλαγές στα footers, κλπ. Όλα τα προηγούμενα είναι θέματα που δουλεύονται αυτή τη στιγμή και δεν είμαι σε θέση να κρίνω για αυτά, οπότε θα πρέπει να τα δει ο καθένας που τα έχει αναλάβει.
- Υπάρχουν κάποια νέα αρχεία στα include που προσθέτουν νέα λειτουργικότητα, με αποτέλεσμα να είναι ίσως εφικτό το refactor από κομμάτια που προσθέσαμε εμείς με τον καιρό στη σελίδα. Ίσως θα ήταν καλό να καταγραφούν μερικά από αυτά, ώστε να τα έχουμε στα υπόψιν μας στο μέλλον. Θα προσπαθήσω να συνεισφέρω σε αυτό κατά την αναλυτικότερη περασιά που θα κάνω στις επόμενες μέρες.
Όλα αυτά που περιέγραψε μόλις τώρα + κάποια άλλα είναι οι λόγοι που βιαζόμουν να κάνουμε review και merge αυτού του PR αλλά ακόμα δεν καταλαβαίνω γιατί κανείς δεν του δίνει σημασία. Θεωρώ πως θα διευκολύνοταν πολύ η κατάσταση αν κάναμε update το theme και μετά δουλεύαμε με αυτήν την έκδοση ως αφετηρία. |
Κυρίως διότι η οργάνωση και συνεργασία μεταξύ τόσων ανθρώπων είναι μάλλον η πιο δύσκολη διαδιακασία! Οι περισσότεροι (όπως κι εσύ) είμαστε assigned σε 5-6 issues και 1-2 pull requests, από όσο έχω δει, με αποτέλεσμα να μην ανταποκρίνονται όλοι τόσο άμεσα. Προσωπικά, χρειάζομαι τουλάχιστον 1 ώρα για να κάνω σωστά ένα review σε issue μέτριας δυσκολίας και σε συνδυασμό με τα παραδοτέα που δουλεύω ήδη, τώρα βρήκα το χρόνο να ασχοληθώ με το συγκεκριμένο... Υπομονή μέχρι να το κοιτάξουν κι οι υπόλοιποι! Φαντάζομαι και κατά τη βαθμολόγησή, θα ληφθεί υπόψιν το που οφείλονται καθυστερήσεις. Οπότε μην ανησυχείς και στο μεταξύ κοίταζε τα issues που είσαι assigned ή το επόμενό σου θέμα. |
|
@JimDragon @p15zerv σίγουρα η νέα έκδοση θα φέρει και βελτιώσεις και προβλήματα, για την ώρα αρκεί να αποφύγουμε κάτι καταστροφικό, π.χ., να μην κάνει μπιλντ η τρέχουσα έκδοση καθόλου. στην πορεία διορθώνουμε τα μικρο-προβλήματα, έτσι είναι όλες οι αναβαθμίσεις σε πολύπλοκα-συνεργατικά έργα |
|
Λοιπόν πρέπει σύντομα να το κοιτάξουμε αφότου κλείσουν τα 2-3 (το πολύ) PR που υπάρχουν στο αποθετήριο του θέματος, και συγγνώμη για την καθυστέρηση εκ μέρους μου. Γενικά θα πρέπει να γίνει ένα re-deploy στο Netlify με την πιο καινούργια έκδοση του site-gr γιατί έχουν γίνει πολλές προσθήκες feature. Αλλά με το που κλείσουν τα PR μέσα στις επόμενες ημέρες θα προχωρήσει με προτεραιότητα. |
There was a problem hiding this comment.
Καλησπέρα ζήτω συγνώμη για την καθυστέρηση του συγκεκριμένου θέματος. @JimDragon
προσπάθησα να τρέξω τοπικά σε καινούριο branch το site-gr αφού το έκανα update με τις τελευτές αλλαγές από το master και με remote them το branch σου . Το site δεν τρέχει διότι έχει μετακινηθεί ο φάκελος _include. Δεν είναι δικό σου λάθος αλλα δικό μας που το αφήσαμε να παει τόσο πίσω .
|
@GIANNIS-AGGELIS ναι γενικά πρέπει να το ξανακάνω update για να είναι ακόμα έγκυρο το pull request. Περιμένω πότε θα πλησιάσει η στιγμή για το merge οπότε θα το ξανακοιτάξω τότε. |
|
Ξαναενημέρωσα το θέμα και επειδή είχε βγει νεότερη έκδοση (4.19.2) σε σχέση με αυτή που παρέχω στο PR το ξανα αναβάθμισα. Τώρα όμως σκέφτομαι άμα είναι να κάνω νεο PR και να κάνω αυτο close για να είναι πιο καθαρό. Διαφορετικά θα πρέπει να κάνω force push στο branch που κανω pull request εδώ και για να πω την αλήθεια με τρομάζει λίγο αυτή η ιδέα. Άμα πατήσετε το live demo link στο πρώτο ποστ θα δείτε την νέα έκδοση 4.19.2 |
ναι, αν σε βολεύει, αρκεί να βάλεις και ένα λινκ εδώ για να έχουμε πρόχειρο το ιστορικό άλλωστε κτυπάει ένα αρχέιο ήδη, ίσως να κτυπήσει και άλλα γιατί υπάρχει ένα τελευταίο αίτημα σε εκκρεμότητα |
|
Θα περιμένω να κλείσει και το #31 και μετά από αυτό θα σας παρακαλούσα όλους να μην κάνετε κάποια άλλη δραστηριότητα στο minimal-ionio και να κάνουμε merge το επαρχόμενο PR με την ενημέρωση του θέματος σε 4.19.2 . Μετά από αυτό θα μπορούμε να συνεχίσουμε όλοι κανονικά. |
|
@JimDragon Περάστηκε και το #31 οπότε μπορείς να προχωρήσεις! |
ghost
closed this
closes #2
Έριξα μια ματιά και σήμερα για παραπάνω bugs / instabilities αλλά το μόνο που βρήκα παραπάνω από το table rendering error ήταν το oversized font που είχε στο people page. Έχοντας λοιπόν διορθώσει και τα 2 bugs και χωρίς να έχει αναφέρει κανείς κανένα παραπάνω νομίζω είμαστε έτοιμοι να το κάνουμε merge.
[Live demo]
[repo link]