I tried my best to include as many merges and commits as possible into the master-backup branch.

@CS2103AUG2017-T10-B3/developers check that all your commits around v1.3 are captured successfully. (With the exception of @yuheng222 PR which causes the app to not open again when data has been changed e6513ed)

For now, we will all have to be extra vigilant when making PRs.w

This branch will be our fall back plan.

You can consider wrapping the address text on the person card as well since long addresses cannot be displayed fully.

...so that I can better manage my contacts when my relationship with that person changes

The app should save the last selected theme so that when the user opens the app again he/she does not have to change the theme via a command again

...so that it feels more like interacting with a person rather than a machine

Perhaps you can specify in the command format as well as other parts of the user guide that tags can only be one word (so that people don't try doing addtag 1 best friend)

Also, adding tags is case sensitive but filtering them is case insensitive. Is this intentional?

Deleting when only one person is displayed in the list (such as find alex followed by delete 1) will cause the person list panel to turn white.

...so that it's easier to locate a person

I want to make the tags in the tagPane behave like the tags in PersonCard i.e adding or removing a tag can be shown accordingly. But it seems that the tag pane remains static even though I implement the bindListener method and I don't know where is the bug:(

Pls help!! T T

It seems to me that if I perform any action that changes the addressbook (edit, removetag, addtag etc) the app cannot be opened again until I delete the data folder

...so that I can obtain the information I need immediately

Welcome screen is a blank white image. Differs from user guide

Right now the NOK name and phone are displayed below the rest of the fields, however it is not immediately obvious what the name and phone refers to. Maybe you can indicate what those fields are with NOK: and NOK: or something like that so its clearer

As per the title, it's extremely erratic on my end. Sometimes when I do see it and I click on it, it brings me to a github page which tells me error 404. I am very sad 😭

I have also tried creating a new PR but github doesn't allow me as it says there already is a PR.

See the video for more details. I have recorded my screen as I keep refreshing the page.

https://www.youtube.com/watch?v=gQaKVEjM8MM

My PR re-appears again near the 20 second mark.

Master branch is failing tests after merging from #34

Pull request passes all tests.

Master build seems to be failing this particular test

guitests.GuiRobot$EventTimeoutException at BrowserPanelTest.java:47

Pull request did not modify the files related to the BrowserPanel or the BrowserPanelTest.

@yuheng222 Are you able to take a look? I think it might have something to do with your Google Maps redirect.

...so that I can update my contact info when needed

Perhaps you can allow undo to undo the changes in theme to make it easier for the user to change themes.

It looks like this on my laptop, not sure if it is a problem on my end.

...so that I can better manage my contacts.

The PersonList panel takes up a bit too much space and results in the map being quite small and hard to read on my laptop. Perhaps you can reduce the size of the PersonList panel as it does not need to be so big

Google maps characters invalid.

...so that I can have more ways to reach the person.

You may consider separating the NOK from the original contact, as the NOK is essentially another contact. For example, set an option/field to link the NOK contact to the original contact.

It may be a good idea for users to click on the contact, and have a pane display more details.

Overall, I think the functions worked pretty well. The excel export is a nice feature. All the best!

...so that I can have a pleasant user experience