dev-core#564: Differentiate case relationships in Contact Summary page

[monishdeb]

Overview

The relationships tab doesn't do much to communicate that a given relationship is tied to a specific case. if you click "more" you can click to manage the case, but from the listing, there is no visual queue to connect it to a case, which makes the relationships look like duplicates.

Before

After

As per this improvement:

1. Adds a case icon against case-relationships.
2. On hover, it opens a modal to show the case relationships info
3. It has a 'Manage Case' link clicking which opens the Case management page in a medium dialog box

Comments

ping @colemanw @lcdservices

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[lcdservices]

we've tested this PR and are already using in production.

[yashodha]

@lcdservices @monishdeb
I tested it with civicase extension. The Manage Case link when clicked doesn't show anything in the popup

.

[lcdservices]

I had tested with core CiviCase, so maybe the issue is specific to the case extension.

[mattwire]

I like the extra icon and the case details on hover. But I'm not sure about the UI experience of having menu items on that same icon. Can the "manage case" link be added with the other links on the right instead?

[yashodha]

@lcdservices yes this is specific to civicase extension

[monishdeb]

@lcdservices @mattwire @yashodha thanks for your feedback. I agree that its not necessary to show 'Manage Case' link in the action board, as we already have that as an action link in the right of a case relationship. This is how it looks now:

[lcdservices]

@monishdeb tested and confirmed with CiviCase core. Looks good.

[yashodha]

@monishdeb looks good to be merged. Can you squash your commits into one so that it is ready to be merged?

[monishdeb]

Thanks @yashodha @lcdservices for your feedback. I have squashed the commits, can you please merge this PR now? Thanks

[seamuslee001]

adding merge on pass based on review

[yashodha]

@monishdeb sure, will merge soon after the tests pass

[seamuslee001]

Jenkins re test this please

[seamuslee001]

Jenkins re test this please

[yashodha]

test this please

[seamuslee001]

Jenkins re test this please

[seamuslee001]

Merging as per the tag