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

Table not refreshing properly #64

Closed
Sembrani opened this issue Feb 1, 2016 · 8 comments
Closed

Table not refreshing properly #64

Sembrani opened this issue Feb 1, 2016 · 8 comments
Labels

Comments

@Sembrani
Copy link

Sembrani commented Feb 1, 2016

Empty cells appear when tutorials which clash with others are selected, when the width of the column is reduced when all clashes in a day are removed, disfiguring the table. Can be fixed with a page refresh.

@samyex6
Copy link
Collaborator

samyex6 commented Feb 2, 2016

Do you remember the order you clicked to trigger that bug?
E.g. COMP1110 -> COMP1710 -> COMP2100 -> Clicked 'Choose' on 1710 Tue 9am tut -> cleared COMP2100 courses.

@Sembrani
Copy link
Author

Sembrani commented Feb 5, 2016

Fastest way I've found to reproduce:

ENGN1217, ENGN1218, ENGN1219 -> Chose ENGN2217 0900 - 1200 Computer Laboratory1 / 1 and ENGN2217 1400 - 1700 Practical1 / 3

@samyex6
Copy link
Collaborator

samyex6 commented Feb 6, 2016

The issue should now be resolved, pls try again 🎉

@Sembrani
Copy link
Author

Sembrani commented Feb 7, 2016

Still an issue.
issue

@samyex6
Copy link
Collaborator

samyex6 commented Feb 7, 2016

ah that, that's just some vacant slots after you delete contents inside those grids, will deal with that later.
but this issue should be fixed:
capture

@samyex6 samyex6 added the bug label Feb 7, 2016
@Sembrani
Copy link
Author

Sembrani commented Feb 8, 2016

awesome. You guys are doing a great job with this website.

@samyex6
Copy link
Collaborator

samyex6 commented Feb 8, 2016

thanks :)

@samyex6 samyex6 closed this as completed Feb 8, 2016
@samyex6 samyex6 reopened this Feb 8, 2016
@samyex6 samyex6 mentioned this issue Feb 17, 2016
@catmanjan
Copy link
Owner

For now users can just refresh the page to remove the artefacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants