New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KQL Parse error look incorrect due to text wrapping in toast notification #49377
Comments
Pinging @elastic/kibana-app (Team:KibanaApp) |
Pinging @elastic/kibana-app-arch (Team:AppArch) |
I think I fixed that as part of a different PR, will check and report back. |
Just checked, this is still an issue. |
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
I checked if it's an easy fix, but I think just enabling newlines in the markdown component doesn't make this error message More useful Also the used Markdown renderer is marked to be deprecated: /** @deprecated use Removing from @elastic/kibana-data-discovery ownership since it needs work on the embeddable rendering, that's not just a little fix |
Pinging @elastic/kibana-presentation (Team:Presentation) |
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
removing |
fyi, this is how we make it work in the Discover error: white-space: break-spaces;
font-family: ${euiTheme.font.familyCode}; |
@kertal can you help me with this? I see this in main, it should be displayed differently? |
@stratoula I can't see if there's a newline in this example ... like here Since the text align is left, and the newline is rendered, the last part of the error, this ascii arrow makes sense. it wouldn't if it was als rendered without newlines, and centered |
@stratoula @kertal It's also necessary to use Misaligned Aligned
|
I'm not sure that this is an issue that we could fix on the Dashboard side. Aren't the errors there are surfaced on the Dashboard arising from the Lens / Saved search Embeddables? |
@ThomThomson have you checked maps? |
@ThomThomson This might not be related to Dashboard directly, but it does seem to be related to embeddables. The issue isn't with the errors themselves, it's the component displaying them which seems to be |
Kibana version: master, haven't checked if it was introduced earlier
KQL parse errors were originally developed to be displayed in the old notification area at the top of the application. This was important because the arrow pointing at the location of the error depended on the text not wrapping. At some point those errors were updated to be displayed in a toast, and now they look broken:
Per the toast guidelines in the EUI docs we should probably display an abbreviated error message in the toast with a button to display the full message including the arrow in a larger modal.
The text was updated successfully, but these errors were encountered: