Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 5 commits into from

Conversation

alexpoulis
Copy link

@alexpoulis alexpoulis commented Apr 14, 2022

Σχετικό Issue

closes ioniodi/sitegr#386

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

  • Η αλλαγή που θα κάνω θα γίνει σε αυτό το repository και συγκεκριμένα θα αλλάξω στην πρώτη γραμμή το class από sidebar__right σε sidebar__right__with__scroll__bar sticky το οποίο είναι ένα νέο class
  • Το class αυτό βρίσκεται εδώ και συγκεκριμένα στην γραμμή 93(Αρχικά ήταν στο minimal mistakes αλλά μετά από παρατηρήσεις σε συμφοιτητές αποφάσισα να το μεταφέρω).
  • Demo για courses & ects-calculator
  • Σχετικοί συνεργατές: @p19kala @gordthrow

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

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

Copy link

@gordthrow gordthrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

δεν βλέπω κάποιο πρόβλημα στο pr σου και με τα αρχεία που έχεις αλλάξει δεν νομίζω να υπάρξει κάποιο θέμα άρα εγώ σου δίνω το οκ.

@alexpoulis
Copy link
Author

alexpoulis commented Apr 21, 2022

@gordthrow @p19kala Μετά από παρατηρήσεις σε συμφοιτητές για αλλαγές μέσα στο minima mistakes έκανα revert το αρχείο στην αρχική του κατάσταση και μετάφερα τις αλλαγές στο main.sccs. Άμα μπορείτε κάντε ένα review ακόμα για τις αλλαγές just in case.

Copy link

@p19kala p19kala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Σοφη αλλαγη και βλεπω οτι συνεχιζει να δουλευει χωρις προβλημα.

Copy link

@gordthrow gordthrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@epidrome
Copy link
Member

αν δείτε προσεκτικά την τεκμηρίωση του θέματος υπάρχουν προβλήματα στην ενημέρωση αν πειράζουμε τα αρχεία του αρχικού θέματος, για αυτό δημιουργήθηκε το minimal-ionio
https://github.com/ioniodi/minimal-ionio/blob/master/_sass/minimal-ionio.scss

@epidrome epidrome closed this May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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