-
Notifications
You must be signed in to change notification settings - Fork 39
Notes from July 21 design meeting
@ UWindsor, Essex Hall, MAC Room 120 Time: 10 am to 1 pm
Attendees: Josh, Neha, Hardik, Stephanos, Matthew, Blandine
-Include instructions in an issue on how to test -Make general set of instructions for complete testing
-Include instructions on how to test all issues -If its a large issue, also provide unit test cases with request (PR?)
Travis is implemented for automated testing on GitHub. Will make it easy to find if a fix breaks the existing code
Selenium Tests? -Could be slowed down by timeouts -Not a priority
-Siri compatibility Open an issue for this [Matthew]
-Might be fixed with new code -New button to "Resend Verification Link" -will have the same interface as "Forgot Password"
-Should be fixed with Hardik's request (PR?)
-Users do not like the display of percentages -When they see percentages they get confused -Change from percantages to stars? -When users are signed in, they can just be given a checkmark or X if a place meets their standards. -Only show them if their options are available rather than all options -Give them Pie charts for only the groups theyve selected on their profile with details -List accessibility criteria with how many people rated it (ex. 12/20) -if over 50% of reviews say it is available, give a green checkmark (and make the rating ratio green as well?) -Otherwise give a red X (and make the rating ratio red as well?) -Total rating will be based off of the average of the checkmarks and X's
-Users who have had their reviews verified can have their reviews carry more weight -Locations they have rated can say "Rated by Verified User"
- Open an issue for this feature [Matthew]
-Only sign them out when they click sign out -Unlikely for multiple users to be using the same device for AL but also deals with this case
-Ratings that are years old should hold less value -Need to figure out how long a rating should hold its peak value before dropping
Open issues for each below ### [Matthew] -change default search radius to 10 km -remove percentages and add -provide sketch/visual of new page [Matthew]
- Open issue on making users stay logged in for 24 hours [Hardik]
- Open issue on making search results start with map and then have the option for table [Matthew] -If user is using a screen reader then it should start with the table
- Open an issue to provide shorter names for accessibility criteria [Blandine]
- Use the shorter names for the profile and the rating view
Open an issue and provide sketch/visual of new page [Matthew] X = [0,49.999] ✓ = [50,100] (number of selected accessibility criteria in brackets) - For Universal view, all criteria are displayed by default. For personal view, only show the categories and criteria selected by the user.
Personal (12) | Universal (58)
|
[See Comments] | [See Comments]
|
[-] Anemities | [+] Anemities
_______________ | [+] Bathrooms
[ニニニニニニニニ］✓ (green) | [+] Elevators
[ニニニニニニニニ］✓ (green) | [+] Entrance
[ニニニニニニニニ］✓ (green) | [+] Parking
[ニニニニニニニニ］X (red) | [+] Public Areas
[more] |
|
[+] Bathrooms |
[+] Elevators |
[+] Entrance |
|