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

[function tests] console like logging api #7028

Merged
merged 2 commits into from
May 4, 2016

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Apr 22, 2016

Blocked by #6955

The common.log and common.debug API currently accept a single string arguments, which is similar to the console.log API but not complete.

This simple change brings updates it to match the API completely, supporting things like multiple arguments and sprintf-style templates.

@LeeDr
Copy link
Contributor

LeeDr commented Apr 22, 2016

I get the same error as Jenkins;

Transformation error; return original code
[TypeError: this[type] is not a function]
SyntaxError: Unexpected token ...
  at exports.runInThisContext  <vm.js:53:16>
  at Object.vm.runInThisContext  <node_modules\intern\node_modules\istanbul\lib\hook.js:174:16>
  at <node_modules\intern\node_modules\dojo\loader.ts:838:14>
  at FSReqWrap.readFileAfterClose [as oncomplete]  <fs.js:380:3>

@spalger
Copy link
Contributor Author

spalger commented Apr 22, 2016

Ah, yes, this will require #6955

@LeeDr
Copy link
Contributor

LeeDr commented Apr 22, 2016

I'll check out #6955 now

The `common.log` and `common.debug` API currently accept a single string arguments, which is similar to the console.log API but not complete.

This simple change brings updates it to match the API completely, supporting things like multiple arguments and sprintf-style templates.
@spalger spalger force-pushed the implement/consoleLikeLogging branch from fb7c71c to d91da04 Compare April 22, 2016 19:09
@LeeDr
Copy link
Contributor

LeeDr commented May 4, 2016

#6955 is in now. I'll check this one.

@LeeDr
Copy link
Contributor

LeeDr commented May 4, 2016

LGTM

@LeeDr LeeDr assigned spalger and unassigned LeeDr May 4, 2016
@spalger spalger merged commit 187882d into elastic:master May 4, 2016
@spalger spalger deleted the implement/consoleLikeLogging branch May 4, 2016 21:42
jbudz pushed a commit that referenced this pull request Sep 27, 2023
`v88.3.0`⏩`v88.5.0`

closes #151514

---

## [`88.5.0`](https://github.com/elastic/eui/tree/v88.5.0)

- Updated `EuiCallOut` with a new `onDismiss` prop
([#7156](elastic/eui#7156))
- Added a new `renderCustomToolbar` prop to `EuiDataGrid`, which allows
custom rendering of the toolbar.
([#7190](elastic/eui#7190))
- Added a new `allowResetButton` prop to
`toolbarVisibility.showDisplaySelector` of `EuiDataGrid`, which allows
hiding the "Reset to default" button from the display settings popover.
([#7190](elastic/eui#7190))
- Added a new `additionalDisplaySettings` prop to
`toolbarVisibility.showDisplaySelector` of `EuiDataGrid`, which allows
rendering extra settings inside the display settings popover.
([#7190](elastic/eui#7190))
- Updated `EuiDataGrid`'s toolbar display settings button icon
([#7190](elastic/eui#7190))
- Updated `EuiTextTruncate` with significantly improved iteration
performance. Removed `measurementRenderAPI` prop, as `EuiTextTruncation`
now only uses more performant canvas render API
([#7210](elastic/eui#7210))
- Updated `EuiPopover` with a new configurable `repositionToCrossAxis`
prop ([#7211](elastic/eui#7211))
- Updated `EuiDatePicker` to support `compressed` input styling
([#7218](elastic/eui#7218))
- Added `gradient` and `palette` icon glyphs.
([#7220](elastic/eui#7220))

**Bug fixes**

- Fixed `EuiPopover`'s missing animations on popover close
([#7211](elastic/eui#7211))
- Fixed `EuiInputPopover` anchoring to the wrong side and missing
shadows on smaller screens
([#7211](elastic/eui#7211))
- Fixed `EuiSuperDatePicker` icon spacing on the quick select button
([#7217](elastic/eui#7217))
- Fixed a missing type in `EuiMarkdownEditor`'s default processing
plugins ([#7221](elastic/eui#7221))


## [`88.4.1`](https://github.com/elastic/eui/tree/v88.4.1)

**Bug fixes**

- Fixed missing `className`s on `EuiTextTruncate`
([#7212](elastic/eui#7212))
- Fixed `title`s on `EuiComboBox` dropdown options to always be present
([#7212](elastic/eui#7212))
- Fixed `EuiComboBox` truncation issues when search is an empty space
([#7212](elastic/eui#7212))

## [`88.4.0`](https://github.com/elastic/eui/tree/v88.4.0)

- Updated `EuiComboBox` to allow configuring text truncation behavior
via `truncationProps`. These props can be set on the entire combobox as
well as on on individual dropdown options.
([#7028](elastic/eui#7028))
- Updated `EuiInMemoryTable` with a new `searchFormat` prop (defaults to
`eql`). When setting this prop to `text`, the built-in search bar will
ignore EQL syntax and allow searching for plain strings with special
characters and symbols.
([#7175](elastic/eui#7175))

**Bug fixes**

- `EuiComboBox` now always shows the highlighted search text, even on
truncated text ([#7028](elastic/eui#7028))
- Fixed missing i18n in `EuiSearchBar`'s default placeholder and
aria-label text ([#7175](elastic/eui#7175))
- Fixed the inline compressed styles of `EuiDescriptionListTitle` to use
a taller line-height for readability
([#7185](elastic/eui#7185))
- Fixed `EuiComboBox` to correctly truncate selected items when
displayed as pills and plain text
([#7193](elastic/eui#7193))

**Accessibility**

- Added `aria-current` attribute to `EuiTablePagination`
([#7186](elastic/eui#7186))

**CSS-in-JS conversions**

- Converted `EuiDroppable` and `EuiDraggable` to Emotion; Removed
`$euiDragAndDropSpacing` Sass variables
([#7187](elastic/eui#7187))

---------

Co-authored-by: Patryk Kopycinski <contact@patrykkopycinski.com>
Co-authored-by: Jan Monschke <jan.monschke@elastic.co>
Co-authored-by: Thomas Watson <watson@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants