Skip to content

[Getting Started Page] Add a space instead of an  #11767

Merged
ycombinator merged 1 commit intoelastic:masterfrom
ycombinator:getting-started-page/space
May 12, 2017
Merged

[Getting Started Page] Add a space instead of an  #11767
ycombinator merged 1 commit intoelastic:masterfrom
ycombinator:getting-started-page/space

Conversation

@ycombinator
Copy link
Contributor

This is a follow up to #11673.

The Getting Started page has two pluggable sections. Users may register multiple strings (markup) into each of these two sections. Currently, we append an   to the end of each string. If the string contains a block-level element, this causes a new line in the UI, which is undesirable.

This PR fixes this problem by using a regular space instead of the  .

That way, a single block-level element before the space does not cause a new line in the UI
@ycombinator ycombinator added :Management bug Fixes for quality problems that affect the customer experience review v5.5.0 v6.0.0 labels May 12, 2017
@ycombinator ycombinator requested review from BigFunger and rhoboat May 12, 2017 20:02
Copy link
Contributor

@BigFunger BigFunger left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@rhoboat rhoboat left a comment

Choose a reason for hiding this comment

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

LGTM. phew that takes care of my quota for the day

@ycombinator ycombinator merged commit 7af3404 into elastic:master May 12, 2017
@ycombinator ycombinator deleted the getting-started-page/space branch May 12, 2017 20:08
ycombinator added a commit that referenced this pull request May 12, 2017
That way, a single block-level element before the space does not cause a new line in the UI
@ycombinator
Copy link
Contributor Author

Backported to:

  • 5.x / 5.5: 122d6cf

snide pushed a commit to snide/kibana that referenced this pull request May 30, 2017
That way, a single block-level element before the space does not cause a new line in the UI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience review v5.5.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants