CDR Tickets

~oseipokuw, I will assume you are talking about changes to the display in XMetaL in this ticket.

I have made changes on DEV to setup the first scenario, choosing SeaShell as the background color

Regarding the second scenario I'm not understanding how this differs from the first scenario.  The background color is set based on the SVPC attribute of the Summary element and the permissions of a user to modify a document are irrelevant.

"The CDR" is not specific enough. The ticket should specify whether it's talking about XMetaL, or some report, and if the latter, which report.

Thanks ~volker Yes, this is a change in the appearance of XMetal. I have reviewed the change and it doesn't stand out well enough but I think it is OK to keep the changes you've made. Are you able to also change the background color of the blocks as well? Please see attached screenshot from DEV. 

Regarding the second scenario I'm not understanding how this differs from the first scenario. 

The second scenario is the same as the first scenario in terms of the outcome but I wanted to make sure not only a summary editor should be able to see the difference but other CDR users should be able to see the difference as well. Or not to create the impression that you need to be a summary 

"The CDR" is not specific enough. The ticket should specify whether it's talking about XMetaL, or some report, and if the latter, which report

Thanks ~bkline I have updated the ticket to clarify the requirements.

I can certainly make the color of the white background stand out more but I don't believe that would be a welcome change for the users.

The advantage of changing the background color of the Summary element is that it included a change for the entire document.  Changes to of the "purple blocks" are limited to specific elements: CdrDocCtl, SummaryMetaData, etc and a color change to those would mostly be off the viewpoint for an editor but I can try to modify those, too.

Is it just the background color you like to have modified or the element's background color and/or label color as well?

I can certainly make the color of the white background stand out more but I don't believe that would be a welcome change for the users.

I agree. I think the Seashell color is OK for now. 

Changes to the "purple blocks" are limited to specific elements: CdrDocCtl, SummaryMetaData, etc and a color change to those would mostly be off the viewpoint for an editor but I can try to modify those, too.

I think changing the purple blocks color of the CDrDocCtl and SummaryMetaData should be good. When a user first retrieves a summary document, the CdrDocCtl and/or the SummaryMetaData block is displayed. That should be enough, the editor does not need to see the change throughout the document to know that it is a SVPC summary.

I have made additional modifications to the legacy summary display for the SVPC documents.

As you can see, the change modifies the document's background color from "white" to "seashell".  It modifies the background color of the elements SummaryMetaData and CdrDocCtl, and it adds a SVPC header to the top of the document:

Thanks ~volker. I think this stands out pretty well. We will review and let you know there are any questions.

The following CSS has been modified to change the display in XMetaL:

• Summary.css
  https://github.com/NCIOCPL/cdr-client/commit/50fb967

• Summary_structure.css
  https://github.com/NCIOCPL/cdr-client/commit/50fb967

This is ready for review on DEV.

Verified on DEV. Thanks!

Due to the attribute name change the following CSS files had to be adjusted:

• Summary.css

• Summary_structure.css

 https://github.com/NCIOCPL/cdr-client/commit/e94ac6a

Verified on QA and PROD. Thanks!