AY1920S1-CS2103T-T17-2 / main

TravEzy
https://ay1920s1-cs2103t-t17-2.github.io/main/
MIT License
0 stars 5 forks source link

Fixed bug when closing window with other page resources and in clear command #180

Closed bjhoohaha closed 5 years ago

bjhoohaha commented 5 years ago

Clear command in Itinerary page not behaving as should

typing clear multiple times produce many windows that may not be the correct behaviour

Screenshot 2019-11-08 at 10 13 06 PM

Bug where error is shown when exiting application before clear window is closed:

photo_2019-11-08_20-46-49

goto itinerary

switch to itinerary window

clear

clear window shows up and before clicking proceed, exit application

exit

error message shows up on intellij

Bug where error is shown when page resources are not closed when exiting application:

photo_2019-11-08_20-46-49

goto address_book

navigate to address book

click help

opens up help window

goto main

navigate to main

exit application by clicking X on window

error message shows up on intelllij

Application's help and code window is not behaving as should

Current application allows the user to open 5 help windows. The application should close all resources in the current page before moving on to get the next page.

Screenshot 2019-11-08 at 10 11 43 PM

pohlinwei commented 5 years ago

Overall, I think that it can be merged (: