[Discover] Use summary column service component for service column

[mohamedhamed-ahmed]

📝 Summary

A new cell renderer for the service.name field was introduced here. As an enhancement we would like to have the cell renderer display the same component as the one used in the summary column for the service.name field, including the APM link and quick filters.

✔️ Acceptance Criteria

• Service name column now displays the same component used in the summary column for the service name field
• Quick filters are displayed
• Service name link is displayed
• The service name link is disabled when APM is disabled

💡 Implementation hints

• Use the ServiceNameBadgeWithActions component to replicate the same behavior

❓ Open Question

• Do we need to keep the Pill shape in the service name column or not?

[elasticmachine]

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

[elasticmachine]

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

[mohamedhamed-ahmed]

@LucaWintergerst @gbamparop are we targetting this for 8.16 ?

Screen.Recording.2024-10-17.at.14.38.59.mov

This is how it will look like after the change, @patpscal @LucaWintergerst wdyt?

[flash1293]

Feature freeze for 8.16 already passed, and I would argue that this is a new feature.

[yngrdyn]

What if we ditch the pill shape? The pill shape is useful in summary column because we have multiple information there and allows users to filter by specific parts inside the column (e.g. host.name, cloud.instance.id, etc)

[patpscal]

Do we need to keep the Pill shape in the service name column or not?

The added functionality of the pill (quick filter in/out) is already solved at a cell level in the service name column, regardless of the content, so that's not adding value.

But there's value in always seeing this parameter displayed in the same way (pill with logo + label), which can help users identify it quickly. Because of this, I would be inclined to keep it as a pill (and maybe remove the on-click behavior of the pill to avoid redundancy with the cell actions), but I don't have a strong opinion.

[mohamedhamed-ahmed]

@yngrdyn I also have mixed opinions there, at the same time it gives kind of an indication that there are more actions when clicked.

[mohamedhamed-ahmed]

and maybe remove the on-click behavior of the pill to avoid redundancy with the cell actions

@patpscal we need the click action for the service name link to navigate there. It is recorded in the demo video at the very end

[mohamedhamed-ahmed]

I am also be a bit more inclined towards keeping the pill shape, or we could also maybe make the service.name itself clickable without the need for the pill shape or the popover with the filters 🤔

[patpscal]

Sorry @mohamedhamed-ahmed, I see what you mean now! Then the more reasons IMO to keep the shape