Skip to content
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

[Security_Solution][Resolver] Style adjustments per UX #71179

Merged
merged 13 commits into from
Jul 13, 2020
Merged

[Security_Solution][Resolver] Style adjustments per UX #71179

merged 13 commits into from
Jul 13, 2020

Conversation

bkimmel
Copy link
Contributor

@bkimmel bkimmel commented Jul 8, 2020

Summary

This PR adjusts style concerns at the direction of the Security Design team per issues enumerated as

from:

find related screenshots / discussion with design team on issue

@bkimmel bkimmel changed the title Style adjustments per UX [Security_Solution][Resolver] Style adjustments per UX Jul 8, 2020
@bkimmel bkimmel added v8.0.0 v7.9.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Resolver Security Solution Resolver feature Team:Endpoint Data Visibility Team managing the endpoint resolver labels Jul 8, 2020
@bkimmel bkimmel marked this pull request as ready for review July 8, 2020 23:44
@bkimmel bkimmel requested review from a team as code owners July 8, 2020 23:44
@elasticmachine
Copy link
Contributor

Pinging @elastic/endpoint-app-team (Feature:Resolver)

@elasticmachine
Copy link
Contributor

Pinging @elastic/endpoint-data-visibility-team (Team:Endpoint Data Visibility)

@bkimmel
Copy link
Contributor Author

bkimmel commented Jul 9, 2020

@marrasherrier provides her 👍 in absentia per Slack convo

title: i18n.translate('xpack.securitySolution.endpoint.resolver.panel.processDescList.path', {
defaultMessage: 'Path',
}),
title: 'process.executable',
description: processPath(processEvent),
Copy link
Contributor

@michaelolo24 michaelolo24 Jul 9, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

double checking, but should the description still be processPath()

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for checking, yeah that's what they call it in ECS:
image

@bkimmel
Copy link
Contributor Author

bkimmel commented Jul 10, 2020

@elasticmachine merge upstream

@bkimmel bkimmel merged commit 0f9c80d into elastic:master Jul 13, 2020
@bkimmel bkimmel deleted the resolver/stylefix-related-detail-panel branch July 13, 2020 13:27
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jul 13, 2020
* master: (28 commits)
  skip flaky suite (elastic#71361)
  [Ingest Manager] Add UI to enroll standalone agent (elastic#71288)
  Node options from cfg file for production (elastic#62468)
  [APM] Improvements to the ML Settings page (elastic#71309)
  add old .chromium to gitignore to prevent it from being accidentally committed
  [Ingest Manager] Simplify add/edit package config (integration) form (elastic#71187)
  Ensure Other bucket works on scripted fields. (elastic#71329)
  [APM] Anomaly detection setup link with alert if job doesn't exist (elastic#71229)
  [APM] Anomaly detection integration with transaction duration chart (elastic#71230)
  inclusive language (elastic#71438)
  [Ingest Manager] During fleet setup create an enrollment for every config (elastic#71308)
  Improvements to our developer guide (elastic#67764)
  [SIEM][Detections] Fixes index patterns order (elastic#71270)
  [Metrics + Logs UI] Add test for logs and metrics telemetry (elastic#70858)
  [Maps] Inclusive language (elastic#71427)
  [Logs UI] Unskip log highlight api integration test (elastic#71058)
  [Security_Solution][Resolver] Style adjustments per UX (elastic#71179)
  [Functional test] Increase the timeout to click new vis function (elastic#71226)
  [Discover] Migrate async import of embeddable factory to actual embeddable (elastic#70920)
  fix overflow (elastic#70723)
  ...
bkimmel added a commit that referenced this pull request Jul 14, 2020
[Security_Solution][Resolver] Style adjustments per UX (#71179)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Resolver Security Solution Resolver feature release_note:skip Skip the PR/issue when compiling release notes Team:Endpoint Data Visibility Team managing the endpoint resolver v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants