syndesisio / syndesis-ui

The front end application or UI for Syndesis - a flexible, customizable, cloud-hosted platform that provides core integration capabilities as a service. It leverages Red Hat's existing product architecture using OpenShift Online/Dedicated and Fuse Integration Services.
https://syndesis.io/
14 stars 28 forks source link

Data mapper integration side panel looks different from the one in the creation flow #973

Closed dongniwang closed 7 years ago

dongniwang commented 7 years ago

Other than the background color is different (white vs. gray), part of the integration side panel in the data mapper is covered by the scroll bar. Also, hovering on connections and steps icon in the data mapper side panel shows no tooltips.

Would it be helpful if we add tooltips to the connections and steps? I know we have that on the integration details page.

cc: @sjcox-rh

image

image

gashcrumb commented 7 years ago

I thought the background was grey in the design, i.e. it's supposed to be a cut-down version of that view so the data mapper has more space.

+1 to having tooltips and yes, it should be adjusted to look ok when a scrollbar is present, especially on Windows which has a wider scrollbar.

dongniwang commented 7 years ago

Hmm...I honestly don't remember if it's supposed to be white or gray :( The background color is less important here. You're right, it's supposed to be a cut-down version. Only found this design on InVision (which has a black background color): https://redhat.invisionapp.com/share/VQ8UZ7PCE#/225286606_IPaaS_DataMapper_Transformations_2-24-17-pdf_3

If it's not too hard to add the tooltips, that would be great!

sjcox-rh commented 7 years ago

Yeah the designs that @dongniwang sent were the latest (back from Feb though) for the stripped down integration visualization on data mapper. It was supposed to be the darker background to differentiate it from the rest of the content on the page but like Dongni said, not super important at this time if it's too time consuming of a change, @gashcrumb

+1 on tooltips!

gashcrumb commented 7 years ago

We should at least try and ensure there's no horizontal scroll bar for starters, @astefanutti any chance you could have a look at this one?

astefanutti commented 7 years ago

@gashcrumb sure, let me have a look at it. Just that on Mac it displays better so hopefully I'll be able to fix it for Windows as well.

astefanutti commented 7 years ago

Should be fixed (scrollbar, background color) with #1012. I've just created #1013 to follow-up on the tooltips.