Commit graph

4894 commits

Author SHA1 Message Date
Aleh Zasypkin 15b272a87a
Provide realm name for OIDC/SAML authenticate requests. (#44984) 2019-09-16 10:32:58 +02:00
Aleh Zasypkin a9f5c2f30a
Enable PKI integration tests. (#44115) 2019-09-16 09:30:19 +02:00
Shahzad 337c76f383
[Uptime] Fix/issue 41924 show permission warning if no right for uptime (#44968)
* add not found page

* update error handling and 404 page

* update unit tests snaps

* reverted a small change

* Resolved formatting issues

* update unit tests
2019-09-14 16:40:02 +05:00
Xavier Mouligneau 19867b9709
[SIEM] Fix timeline persistence against url and events viewer (#45657)
* fix timeline  persistence against url and events viewer

* fix reset fields browser for events view

* fix inspect event

* review I

* review II
2019-09-13 20:19:46 -04:00
Nate Archer 334506a96a
[DOCS][CODE]: Fix TOC issue for renamed Code page (#45700)
* [DOCS] Fix beta tag in Code Docs

* Fix TOC issue for renames Code page
2019-09-13 16:40:19 -05:00
Frank Hassanabad c9a11ee971
[SIEM] Adds process_started ECS fields to Uncommon Process Table (#45664)
## Summary

Adds process_started ECS fields to the uncommon process table for any agents which support it. This removes duplicates from the uncommon process table as well in some cases where we were counting process started and process ended as two different counts.

https://github.com/elastic/kibana/issues/45530

Anyone outside of beats implementing this will need these two fields to show up:

```ts
event.type: process_start
event.category: process,
```

Their ECS record should have this bare minimum:

```ts
  "event": {
    "type:": "process_start",
    "category": "process",
  },
```

To show up
### Checklist

Use ~~strikethroughs~~ to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, [including a check against IE11](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility)
- [ ] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)
- [ ] [Documentation](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#writing-documentation) was added for features that require explanation or tutorials
- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
- [ ] This was checked for [keyboard-only and screenreader accessibility](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Accessibility#Accessibility_testing_checklist)

### For maintainers

- [x] This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)
- [x] This includes a feature addition or change that requires a release note and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)
2019-09-13 14:37:52 -06:00
spalger 5696b67c92 [Reporting] Skip failing test (#45656)
(cherry picked from commit 40e24547ae)
2019-09-13 13:36:11 -07:00
Lee Drengenberg d9cd4e614e
[Reporting] skip flaky reporting tests (#45659)
* Add option for Internet Explorer IE11

* WIP trying to figure out IE options/capabilities

* more attempts to get IE tests working

* more work, but still can't set font size in Dev Tools Console

* two changes for IE

* revert skip in console test

* this change *appears* to set absolute times but they don't stick

* Shakespeare passing on all 3 browsers

* add config.ie.js, change 'internet explorer' to 'ie'

* increase notifications:lifetime so ew have a chance to see the message

* correct path with this missing slash

* skip flaky tests issues #45499 #42725

* revert these files to master

* revert to master
2019-09-13 15:34:35 -05:00
Yuliia Naumenko c6e78599e8
Create unit test for registerBuiltInActionTypes and move common code (#45648)
* Create unit test for registerBuiltInActionTypes and move common code
2019-09-13 13:16:40 -07:00
Mike Côté 68ffe7e18b
Update alerting docs to mention manage_api_key cluster privilege requirement (#45375)
* Update alerting docs to mention manage_api_key cluster privilege requirement

* Update docs
2019-09-13 15:42:02 -04:00
Tim Sullivan 9a56d426ed
[Reporting/ESQueue] data read/write failures are error logs (#45478)
* [Reporting/ESQueue] data read/write failures are error logs

* fix warning level
2019-09-13 11:39:31 -07:00
Nate Archer 84a56d580a
[DOCS][CODE]: Rename "Getting started with Code" to "Import your first repo" (#44924) (#45471)
* [DOCS] Fix beta tag in Code Docs

* [DOCS][CODE]: Issue #44924 - Rename Get started to Import your first repo
2019-09-13 13:33:09 -05:00
Tim Sullivan 4a9dc664c6
[Reporting/Typescript] Fix types for JobParamsPDF, BrowserConfig, ChromiumDriverFactory (#45209)
* [Reporting] Correct PDF Params type definition

* Fix types for BrowserConfig, ChromiumDriverFactory

* remove jobparams type

* spot fixes

* fix guard in compat shim

* fix legacy functional test

* fix jest test
2019-09-13 10:54:41 -07:00
Justin Kambic 271be95171
[Uptime] Add TLS cert expiry to GQL schema for individual docs (#44506)
* Add certificate expiration date to uptime GQL schema.

* Update test fixture/snapshot for broken unit test.

* Extract TLS data to two separate GQL types.

* Remove unused field from schema/adapters.

* Update broken functional tests.

* Extract TLS data to two separate GQL types.

* Remove unused field from schema/adapters.

* Update broken functional tests.

* Modify schema/types and parsing code to avoid weird data structure.

* Fix busted test snapshot/mock data.
2019-09-13 12:52:15 -04:00
Alison Goryachev 7e359276be
Address accessibility issues in management apps (#45024) 2019-09-13 12:14:26 -04:00
Ryland Herrick 79d531cecc
[Code] Results change with search context (#45545)
* Clean up test setup

Removes a redundant assignment.

* Update search suggestions after scope changes

Without the additional call to this.updateSuggestions, the autocomplete
suggestions remain from the previous scope and no action is taken until
the user submits the search form.

This also adds a null check on the `onSearchScopeChanged` prop.

* Test search scope change functionality

Adds a basic test around the search scope functionality in the QueryBar
component.

Unfortunately, due to the fact that the searchScope prop is actually
controlled in a parent, this test is of questionable utility.
2019-09-13 10:50:41 -05:00
Walter Rafelsberger d8cc11d87d
[ML] Single metric viewer: Fix race condition when changing jobs. (#45420)
Fixes a race condition where changing jobs in Single Metric Viewer could result in the focus chart being empty.
Applying the job change would cause two events to be triggered which would cause the Single Metric Viewer to refresh twice: The updated time range would be triggered via timefilter, the updated jobs via the globalState listener.
This PR solves the problem by setting a global flag skipRefresh which allows us to skip updating single metric viewer and only update again once both the new time range and jobs are set. The PR also adds some checks to avoid unnecessary updates in d3 code triggered by React render calls.
2019-09-13 07:36:26 -07:00
Joe Reuter 6763836bc7
[Graph] Settings EUI-ification (#44587)
* create graph listing page

* clean up app folder

* remove inline loading menu

* also add badge to workspace route

* fix tests

* fix graph spaces functional test

* generate documentation for new breadcrumb property

* fix test subject names

* remove unused translations

* start implementing save modal flow for Graph

* fix spaces functional test

* wip save modal

* wip save modal

* add and style save modal

* add placeholder to description field

* disable dirty check on breadcrumb navigation and fix delete function

* reactify most of settings

* improve onClick typing on breadcrumb

* fix newline error and use new types in dashboard app controller

* EUIify drilldown

* EUIify drilldown

* fix translation errors

* fix i18n translation for real

* fix icon and work on autoreplace template logic

* use switch instead of checkbox

* code review

* fix i18n phrases

* remove fragments

* add KQL encoder to be able to drill down to discover

* fix flyout overflow issue

* delete old settings logic

* remove obsolete files

* fix tests

* fix i18n phrase typo

* fix most translations

* start implementing review comments

* continue implementing review

* continue cleanup of settings forms

* remove unnecessary max-width and add commentary

* move to async syntax

* clean up implementation

* use description instead of title

* work on settings form

* fix snapshot

* fix drilldown list

* add tests for use_list_keys

* rename stuff and add documentation

* fix translatiojns

* fix some rename references

* start typing saving and loading

* typing persistence stuff

* adress review comments and set width for all save modals

* type serializing logic

* fix bug and improve typing

* fix classname

* wip

* wip

* add notes

* style blacklist group

* remove merge marker

* improve icon rendering

* wip

* start refactoring

* remove js implementation

* remove js implementation

* remove duplicate file

* add some tests

* Design cleanup

* Fix SASS files

* revert style fixes in app.js

* implement todos

* fix errors and prematurely enabling inspect panel

* deselect icon after clicking again

* fix types

* make icon list accessible

* clean up and add some more tests

* improve number handling

* fix deserialization

* polishing drilldown-ui

* rename types

* clean up imports

* fix tests

* add handling for canEditDrillDowns setting

* propagate changes to workspace client

* align tabs with other elements
2019-09-13 15:30:58 +02:00
Walter Rafelsberger ea4900b4c6
[ML] Single metric viewer: Fix cell selection flickering. (#45500)
- Fixes a flickering effect when selecting cells in the Single Metric Viewer's swimlane. Previously, during selection, dragSelect took care of highlighting cells. The dragSelect-selection would then be cleared and the d3-based selection only took over after new data was loaded which resulted in the selection not being visible during loading. Now, the dragSelect-selection is only cleared immediately if only cells not containing any anomalies were selected.
- Also fixes a bug where only the first and last cell in the overall swimlane would be highlighted instead of the full range of selected cells.
2019-09-13 06:18:55 -07:00
Devin W. Hurley 8bc8ae2053
[SIEM] Enable eslint rule for duplicate import statements (#45552)
* enable eslint rule for duplicate import statements

* updates lint warn to an error
2019-09-13 09:15:42 -04:00
Walter Rafelsberger ec5d1a9ce9
[ML] Fixes getTab() to always return an array. (#45616)
- Fixes getTabs() to always return an array. I was wondering why TypeScript didn't flag the tabs.map(...) part because tabs could possibly be undefined. It's because Record assumes that every key exists. More on that can be found in this blog post.
- This PR fixes a) the type of TAB_MAP by wrapping it in Partial<...> so it correctly flags the tabs.map() part with tabs possibly being undefined and b) fixes getTabs()'s return value by always returning an array.
2019-09-13 05:39:38 -07:00
Xavier Mouligneau f5e91da99c
[SIEM] Fix paginated table state when switching tab (#45487)
* Keep state of host/netwrok table when switching tab

* keep state in the event viewer when switching tab in host page

* refactor test to ensure activePage updates from prop change

* rm comments

* add/fix unit test

* fix i18n

* need to rm weird files

* rm weird files

* fix inspect button for real + loading view between tabs

* cleanups

* commented by mistake

* fix logic loading

* review I

* review II

* review III

* review IV
2019-09-13 07:22:27 -04:00
Maryia Lapata 1223ab97a6
[Vis: Default Editor] EUIficate 'Metrics & Axis' tab (#43772)
* EUIficate pointe-series and grid

* Apply TS

* Show grid on a panel

* Remove extra space

* Add TS

* Use BasicOptions

* Adjust func test

* Add dataTestSubj prop to SelectOption

* Use id instead of data-sest-subj

* Disable show x-axis lines when there is histogram agg

* Add tooltip for disabled 'Show x-axis lines' config

* Add series_options.tsx

* Migrate series

* Improve changeValueAxis

* Create CategoryAxisPanel

* Create ValueAxesPanel

* Create CustomExtentsOptions

* File renaming

* Use TruncateLabelsOption

* Set min value

* Add validation

* Refactoring

* Move types upper

* Convert series logic

* Convert valueAxes logic

* Rename function

* Add validation and refactoring

* Adjust styles

* Fix rotates

* Apply config for other vis

* Remove old directives

* Fix useHook usage

* Update vislib_vis_type.js

* Fix

* Move Threshold line panel into a separate component

* Remove unused translations

* Apply ui suggestions

* Update functional tests

* Remove angular unit test

* Refactoring

* Fix dependencies

* Refactoring

* Add validation

* Add validation for other vis

* Refactoring of onPositionChanged handler

* Refactoring of addValueAxis

* Get rid of a useEffect

* Refactoring of updateAxisTitle

* Update useCallback

* Refactoring

* Refactoring of collections config

* Refactoring of setChartValueByIndex and setValueAxisByIndex

* Update metrics_axes_options.tsx

* Watch current tab for correct accordion height

* Revert axis title logic

* Fix axis id and name number

* Fix code review comments

* Fix functional tests

* Update visualize_page.js

* Move option tabs in common

* Fix code review comments

* Update index.tsx

* Refactoring

* Fix merge conflict

* Show ThresholdPanel when it's configured

* Fix passing a current tab to vis-options

* Localize 'Count' text

* Pass one axis to CategoryAxisPanel

* Fix y-axis name number when x-axis position changed

* Fix Y-axis name number when position Y-axis changed

* Fix refresh loop

* Re-organize controls

* Apply code review comments

* Refactoring

* Fix extents empty value

* Update y_extents.tsx

* Fix code review comments

* Fix updating several seriesParams when removing axis

* Add thresholdLine config to horizontal bar

* Refactoring

* Reset grid.valueAxis when the axis deleted

* Fix refresh loop

* Set interpolate config for line

* Enable "show dots"
2019-09-13 14:12:43 +03:00
Mike Place 384d2200ce
Respect xpack.monitoring.show_license_expiration (#45537) 2019-09-13 06:42:08 +00:00
Garrett Spong 42b80af630
[SIEM] Moves installation of jobs from initial page load to opening of Anomaly Detection UI (#45536) 2019-09-12 14:51:17 -06:00
Nathan Reese bdc2575d79
[Maps] fix adding filters in map embeddable (#45187) 2019-09-12 14:40:41 -06:00
Catherine Liu 433d5b65de
[Canvas][i18n] Keyboard Shortcuts and Help Menu (#45056)
* Translated shortcut descriptions

* Added i18n to HelpMenu and KeyboardShortcutDocs components

* fixed i18n errors

* Removed unnecessary type

* Updated snapshot

* Adds a description for shortcut separator in the KeyboardShortcutsDoc
2019-09-12 12:07:43 -07:00
James Gowdy b5fc70e013
[ML] Adjusting module jobs model memory limit (#45502) 2019-09-12 18:12:40 +01:00
Robert Oskamp bd8c09e63b
Fix model bytes validation (#45508)
This PR adds `model_bytes` to the list of count details that are only validated to be present and re-enables the tests.
2019-09-12 17:17:21 +02:00
CJ Cenizal 7506c7fd76
Remove index lifecycle phase filter from Index Management because the ES API doesn't provide the required information for us to implement the expected behavior. (#45486) 2019-09-12 07:58:38 -07:00
Nathan Reese 24ad526e6c
[Maps] set mapbox Popup maxWidth (#45253) 2019-09-12 08:37:07 -06:00
Alison Goryachev 43c033d942
[ILM] Delete min_age field if warm phase on rollover is enabled (#45412) 2019-09-12 10:26:52 -04:00
Søren Louv-Jansen 15b5839fbd
[APM] Remove transaction.result filter from Errors overviews page (#45379) 2019-09-12 15:10:42 +02:00
Melissa Alvarez 9e3a3dc7c6
[ML] Reorganize ML navigation with top and sub level tabs (#45220)
* Add main nav tabs with sub tabs for new nav

* move transforms to top level main nav

* Make top nav normal font weight

* Update breadcrumbs to take top nav into account

* proper spacing when settings selected

* fix localization error

* Fix functional tests. Update breadcrumbs

* revert analytics breadcrumb update. save for follow up

* ensure main/sub tabs align left

* update dataVisualizer breadcrumbs

* update typescript for tabs
2019-09-12 09:09:36 -04:00
Liza Katz a45b0a7bbf
Split data and expression plugins (#45125)
* split data plugin from expression plugin

* added expressions as dependency to visualizations

* ui/agg import

* expressions start contract
replace __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED with __LEGACY

* Rename contract

* vis deps

* Fix build pipeline mocks

* Added expressions plugin to karma mock

* update imports

* export types from common

* Restore types.ts

* Remove unused expressions plugin server code
2019-09-12 15:57:58 +03:00
Larry Gregory 3e960e3e5c
Spaces - fix SecureSpaceMessage visibility (#44829)
* fix SecureSpaceMessage visibility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
2019-09-12 08:49:12 -04:00
Søren Louv-Jansen 19837fea23
[APM] Add "Quick links" to test plan 2019-09-12 13:23:32 +02:00
Søren Louv-Jansen 8a6b1518ef
Renamed tested-after-ff to apm-test-plan-done 2019-09-12 13:13:14 +02:00
Gidi Meir Morris 1c6b9336e3
[alerting] Renames the AlertInstance.fire to scheduleActions (#45245)
Renamed the misleadingly named AlertInstance.fire to scheduleActions, as it does not in fact "fire" the actions, but rather schedules them to be executed by the Task Manager at the next available opportunity.
2019-09-12 09:34:24 +01:00
Dmitry Lemeshko a33909a480
Change data-test-subj selector logic to support value with spaces (#43682)
* update test-subj-selector

* update locators

* adjust whitespaces removal

* fix another locator

* fix ML test locators
2019-09-12 09:48:09 +02:00
Andrew Goldstein 40147fa83f
[SIEM] Apply highlighting to the Timeline data providers drop area and flyout button (#45173)
## Summary

To indicate they accept droppable data providers, this PR applies highlighting to the Timeline flyout badge and data providers drop area.

Tested in dark and light mode in:
- Chrome `76.0.3809.132`
- Firefox `69.0`
- Safari `12.1.2`

## Timeline data providers (drop area) highlighting behavior

- While a data provider is being dragged, in a page or from within the timeline itself, apply `euiColorSuccess` to the timeline data providers drop area's dashed borders and "plain" text, but NOT to the other primitives, i.e. data provider badges, `AND` / `OR` circular badges, lines, etc  
- While a data provider is being dragged, in a page or from within the timeline itself, but NOT hovering over the data providers drop area, apply the `euiColorSuccess` color with `10%` alpha channel to the background of the data providers drop area
- While a data provider is being dragged over the drop target area inside the data providers drop zone, apply the `euiColorSuccess` color with `20%` alpha channel to the background of the data providers drop area
- While a data provider is being dragged over the drop target area inside the data providers drop zone that will add the data provider as an `AND` clause, apply `euiColorSuccess` color to the dashed border, and fill the background with the `euiColorSuccess` color with `30%` alpha channel to the background of the data providers drop area
- Do NOT apply highlighting styles when a column or field (non-data provider) is dragged from the timeline or the `Events` widget

### Dark mode

![timeline-highlighting-dark](https://user-images.githubusercontent.com/4459398/64755303-ebcb1680-d4e7-11e9-8452-0f91ddae85d1.gif)

### Light mode

![timeline-highlighting-light](https://user-images.githubusercontent.com/4459398/64755318-fc7b8c80-d4e7-11e9-8621-a7eb918468aa.gif)

## Flyout button highlighting behavior

- The (default) timeline flyout button is styled as non-filled EUI button
- The button text reads `Timeline ^` (with a trailing caret), instead of `T I M E L I N E`
- Hovering over the flyout button while NOT dragging underlines the flyout button text, but does not fill the background
- When a user starts dragging a data provider, apply `euiColorSuccess` to all visible flyout button properties (e.g. border, text)
- While a data provider is being dragged, in a page or from within the timeline itself, but NOT hovering over the flyout, apply the `euiColorSuccess` color with `10%` alpha channel to the background of the flyout button
- While a data provider is being dragged, in a page or from within the timeline itself, AND hovering over the flyout such that it's in a droppable state, apply the `euiColorSuccess` color with `20%` alpha channel to the background of the flyout button
- While a data provider is being dragged, in a page or from within the timeline itself, AND hovering over the flyout such that it's in a droppable state, replace the `Timeline ^` with a circle containing a `+` plus sign
- Apply `EuiNotificationBadge` styling to the data providers badge count, and position it in the upper left-hand corner
- Do NOT apply highlighting styles when a column or field (non-data provider) is dragged from the timeline or the `Events` widget

### Dark mode

![timeline-badge-dark](https://user-images.githubusercontent.com/4459398/64755512-5ed48d00-d4e8-11e9-9be2-02da70ac4e16.gif)

### Light mode

![timeline-badge-light](https://user-images.githubusercontent.com/4459398/64755531-7449b700-d4e8-11e9-9239-62ee78a6e652.gif)

Resolves https://github.com/elastic/siem-team/issues/457
2019-09-12 01:04:20 -06:00
Spencer 9a109f2170
Revert "Revert "Revert "Revert "[ci] compress jobs for CI stab… (#45454)
* Revert "Revert "Revert "Revert "[ci] compress jobs for CI stability" (#44584)"""

This reverts commit 148b8c0f90.

* sync changes with Jenkinsfile
2019-09-11 15:27:43 -07:00
spalger c0dd0ca7d3 skip flaky test (#45450) 2019-09-11 14:21:33 -07:00
spalger 45711a3005 skip flaky test (#45447) 2019-09-11 14:20:54 -07:00
Jonathan Budzenski f8abcc4ea3
[test/x-pack] add matching functional_test_runner script (#44939)
* [test/x-pack] add matching functional_test_runner script

* make readme consistent
2019-09-11 14:52:43 -05:00
Jean-Louis Leysens 78cc334c2a
Update dependencies (#45133) 2019-09-11 21:39:13 +02:00
Dario Gieselaar bf7658b431
[APM] Fix IE layout issue for filters popover (#45370) 2019-09-11 21:35:15 +02:00
Dario Gieselaar 0b98cce683
[APM] Use EUITab classNames to render tab links (#45371) 2019-09-11 21:34:43 +02:00
Brian Seeders 27d23c4184 Jenkins pipeline with parallel cigroups (#45285)
* Pipeline

* WIP some work for parallelization with ciGroups

* Fix xpack kibana install dir, and add some debugging

* Attempt to quick fix a few tests

* Revert "Revert "Revert "[ci] compress jobs for CI stability" (#44584)""

This reverts commit 078ac2897f.

* Recombine test groups, and try runbld again

* Mostly cleanup, and fix failed_tests reporting to hopefully work for both pipeline and non-pipeline

* Fix typo in shell script

* Remove some debug code

* Add support for changing es transport.port during testing via TEST_ES_TRANSPORT_PORT

* Fix test that uses hard-coded es transport port and add it back in to parallel groups

* Disable checks reporter again for now

* Set env var for TEST_ES_TRANSPORT_PORT in pipeline

* Update Jenkinsfile for shorter testrunner labels

* Fix another hard-coded transport port

* Fix a new test with hard-coded URLs

* Jenkinsfile cleanup and fix one of the groups

* Fix double slash

* Testing vault credentials on jenkins server

* Add a non-existent credential

* Revert "Add a non-existent credential"

This reverts commit 0dc234c465a5483b1a994cb510a182fef766e9cc.

* Try github-checks-reporter again

* github-checks-reporter should only run for elastic/kibana, forks won't work

* Clean up some debug code

* Changing names around to try to make BlueOcean UI a little better

* Add more stages

* Make some changes to stage structure to mirror a nested example from CloudBees

* Handle TODOs, and some cleanup in Jenkinsfile

* Pass GIT_BRANCH when started without GHPRB, fix branch check

* Fix mailer problem and add code that ensures all tests are in cigroups back in

* Test adding worker/job name to junit report paths

* Remove some duplication from ci_setup scripts

* Fix unit test that uses junit path

* Don't reinstall node every time setup_env is run

* Fix yarn install logic

* Fix another unit test that uses junit output dir

* Download latest ES snapshot after kibana builds

* Make sure junit reports are always processed

* Add two failing tests for testing purposes

* Add support to Jenkinsfile for kibana build e-mails

* Remove some debug code for email sending

* Change JOB env handling in junit paths and move it to a sub-directory

* Revert "Add two failing tests for testing purposes"

This reverts commit 5715203e26922a93483feb0ebb8bb3fdcc3daf8c.

* Fix junit report path in test

* Don't send kibana emails on build abort

* Address PR feedback, formatting and use built-in url formatting library

* Fix path formatting for functional test

* Add email sending back in to Jenkinsfile

* Fix another unit test with path problem
2019-09-11 11:58:28 -07:00
Ryland Herrick 98b652b484
[Code] Search results are fixed to top of page (#45182)
* Fix main search bar to top of screen

By allowing overflow on the main content, we implicitly fix the sibling
search bar in position.

elastic/code#1606

* Allow search suggestion text to be truncated

This gives the parent element a width, so that our text-overflow
rules apply to these elements.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1086218#c4 for details
on the issue.

* Fix alignment issue with overflowed search suggestions

If both the icon and the text are allowed to grow/shrink as needed,
flexbox will grow truncated text based on its original width, leading to
subtle sub-pixel alignment issues with other rows.

By fixing the icon to a specific width, we can ensure that the
suggestion text does not grow too big.

* Remove unneeded CSS

We don't need to flex these elements currently as they contain either no
children or inline elements only.

* style: prefer camelCase element names over hyphens

* Fix search result suggestion overflow on IE

Without this directive on the parent, it had no width or flex growth and
didn't respect the parent width. This allows it to take on the correct
flexed width, and thus take on the overflow rules.

* Remove unused CSS rule

We have an overriding align-items declaration immediately after this
one.
2019-09-11 13:24:31 -05:00