Dqt improvements define fields

[marandmart]

Brief summary of changes

Updated placement of UI elements

• Removed redundant labels
• Updated the sync field label to make easier the understanding of what it does
• Updated Available Fields label to Select a Field to make more clear what the user needs to do
• Hid the previously named Default Visits dropdown field until the user clicks the Sync visit selection checkbox
• Height limited the Sync Visits selection box with a scrollbar so it won't take too much space when there are too many visits available
• Updated filter box look to make it more distinct from the other selection fields
• Changed filter box and add all / remove all button placement to better group them near the available fields and make their relation clearer
• Updated selected field list item so that visual elements won't clash together under certain screen sizes or depending on the amount of text.
• Height limited the visit selection with a scrollbar within a selected field list item in the field list so it won't take too much vertical space when there are too many visits available

Previously

Updated

Link(s) to related issue(s)

558: https://github.com/aces/CBIGR/issues/558

[driusan]

@marandmart sorry it took so long but I was waiting for the translation of the dataquery module to be merged. Can you rebase this and then I'll test it?

[CamilleBeau]

This isn't very important but I think it's a little strange that "Select a field" is smaller than "Selected fields"

Personally I think we do need a visit selection before syncing visit fields in case different visits should be selected for different fields

There are some conflicts to resolve as well

Otherwise it looks great!