wpviews: style them differently

[iseulde]

At the moment it's very difficult to see whether a view is selected when the content inside takes up the whole space. It might be better to use a dashed "outline" with an offset.

I would also move the buttons outside the view so they don't have to move for smaller views.

We could set the color of the dashed outline to the color of the text (with an opacity) so that it has a good contrast for all kind of backgrounds.

Screenshots and patch coming.

[helen]

I like this direction, but I'm just not a fan of centering the content in the editor like that. It's been throwing me off in the experiments on #28328 because I already have a set expectation based on how things have worked up until now. That isn't to say that we can't make changes, and my opinion is but one, but this feels bigger than one might think.

azaozz tried something in the past where the background color of the editor outside of the content area width was different and delineated, and centering the content area, like you'd see in a word processor. Doesn't solve the issue of reserving space for the toolbar if it's to the side though. That space also causes some horizontal scroll at narrower widths, which is the bane of my existence in text editing.

[iseulde]

I personally do like centring the content, because it feels less crammed. Most content in editors seems to be centred and the content in DFW is also centred. But that's also just my opinion. :)

That space also causes some horizontal scroll at narrower widths, which is the bane of my existence in text editing.

It shouldn't cause a horizontal scroll bar. With what theme and content did you test? Was it media overflowing?

[SergeyBiryukov]

Replying to helen:

azaozz tried something in the past where the background color of the editor outside of the content area width was different and delineated, and centering the content area, like you'd see in a word processor.

Related: #21536

[iseulde]

The least we could do here for 4.0 is adding some space between the border and the view.

[iseulde]

:before is used in the patch because outline-offset is not supported in IE. :(

[helen]

No consensus yet, so punting from 4.0. Definitely want to keep refining this over time.

[iseulde]

The buttons have now changed, but the border/background could still be better.

Related: #28588.

[azaozz]

That was a good idea, but... is now superseded by the block editor. The UI there looks much better :)