Embed visualizations become flex.

[dairyo]

What type of PR is this? (check all applicable)

• Bug Fix

Description

Embed visualizations are not flex for vertical but I think they should be flex.
The reason is following less code:

redash/client/app/assets/less/redash/query.less

Lines 211 to 214 in a16f551

	.embed__vis {
	display: flex;
	flex-flow: column;
	}

This code intends that embed visualizations should be flex for vertical.

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Before fix. Not flex version

After fix. Flex version

[arikfr]

@gabrieldutra does this need any changes following the merge of the React version of the viz embed page?

[gabrieldutra]

@gabrieldutra does this need any changes following the merge of the React version of the viz embed page?

No, looks like it works post-merge without changes

[guidopetri]

@shinsuke-nara , thanks for the PR! We've updated a lot of things now that we're Community-driven so - if you're still interested in getting this merged - would you mind rebasing off master to re-run the CI, as well as updating merge conflicts?

We're trying to clean up our PR todo list, so if you're not interested, that's fine - we'll close the PR in about a week if we don't hear back. If you're interested in reopening the PR afterwards, we would also very much welcome that.

[justinclift]

The merge conflict looked pretty simple, so I just made a first attempt at resolving it.

Lets see if our CI tests seem happy with things.

[codecov]

Codecov Report

Merging #4422 (d73bc60) into master (d5b821e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4422   +/-   ##
=======================================
  Coverage   59.71%   59.71%           
=======================================
  Files         151      151           
  Lines       12261    12261           
  Branches     1658     1658           
=======================================
  Hits         7322     7322           
  Misses       4733     4733           
  Partials      206      206           

[justinclift]

This looks like it was already reviewed when first submitted, so now that it's passing CI we might as well merge it. 😄