Teammates: Missing loading icons and bugs

[ccyccyccy]

Missing loading icons for Instructor Session Management page

Instructor Sessions Page

• All buttons under "Deleted Feedback Sessions" table
• Under new session form: "Copy from previous feedback sessions" button
• Under active session table
  
  
  
  • Delete button
  
  
  • Copy button
  
  
  • Remind button: Both on click and after confirming remind
  
  
  • All buttons under "Results" dropdown menu
  
  

Instructor Session Edit Page

• Session Edit Form
  
  
  
  • Clicking save at the top of the session edit form does not show loading icon at the upper save button. Only the lower "Save Changes" button have a loading icon
  
  
  • Delete button
  
  
  • Copy button: Both on click and after confirming copy
  
  
• Question Edit Form
  
  
  
  • Delete button
  
  
  • Duplicate button
    
    
    
    
    
    • Loading icon appears on the "Save Changes" button instead, when the duplicate button is clicked
    
    
    
  
  
  • Loading icon on save is too close to the tick icon on the save button
  
  
• Add New Question -> Choose from template question

[madanalogy]

Raising priority to medium because when the loading time gets long, there is nothing stopping the user from navigating away or clicking other buttons thinking nothing is happening.

[wkurniawan07]

Raising priority to medium because when the loading time gets long, there is nothing stopping the user from navigating away or clicking other buttons thinking nothing is happening.

This has nothing to do with whether there is loading icon or not?

[ccyccyccy]

I think it's more like the user is more likely to navigate away if there is no loading icon.

[Rishika1416]

we are a team of of developers , we would like to work on this issue #10569 . can we proceed?

[madanalogy]

we are a team of of developers , we would like to work on this issue #10569 . can we proceed?

Sure, you're welcome to give it a go. I would recommend separate PRs for individual components, you can tag them as "Part of" this PR (see process.md). Let us know if you need any help.

[jfrairigh]

The "View Results" button under "Results" drop down menu seems to already have a loading icon for when the View Results page populates. Will the spinner that is already present suffice or should I had another one?

[jfrairigh]

Problem Introduction:
I have solved the bullet points under the session edit form. However, there is one component test (BuildPropertiesTest) that isn't passing. It doesn't pass in either my local master or my local working branch so I think it may be an environment issue.

What I have tried to fix the failing test:
I noticed some of the "build.template.properties" files had additional content that the "build.properties" files didn't have so I copied the template files and named the copy "build.properties". However, the test still fails.

Does anyone have any insight that will help me with this?

Here is a screenshot of the test failing:

[madanalogy]

@jfrairigh it's best to create a new issue in the tracker with your request for help, that way we can discuss it there and close the issue after your problems have been resolved

For now you can try running Step 3 of https://github.com/TEAMMATES/teammates/blob/master/docs/setting-up.md and if this doesn't resolve your issue just create a new issue in the tracker yea

[jfrairigh]

I'm currently working on the remainder of the bullet point items for this issue. One of the items under Question Edit Form states, "Loading icon on save is too close to the tick icon on the save button". Here is a screenshot of how it looks currently:

How would you like it to look? To me, it currently looks the same as the spinner alignment on the buttons in the Session Edit Form.

[madanalogy]

I believe that portion has been addressed in a separate PR actually. @damithc current spacing okay? Between the tick and the loading icon in the above screenshot.

[damithc]

I believe that portion has been addressed in a separate PR actually. @damithc current spacing okay? Between the tick and the loading icon in the above screenshot.

My expectations was the spinner will appear in place of the tick, and will be replaced by the tick when the work is done. Would that be better? That way, the button size can remain the same. OK if it's too late to go that way (i.e., if a different approach has been taken in other places already).
Plus, I would like to see a gif or a screen recording of how it actually appears in action.