Tips for Text format filters have excessive left|right padding on narrow screens

#1: filter-tips-narrow-1902872.patch queued for re-testing.

It's not clear to me what the issue actually is. When I added the the padding from the patch to the tips class (via Firebug and/or in Chrome's Developer tools) I didn't see any change, even for very narrow viewports.

@jmarkel, I do see it as the screenshots show the before (in issue summary) + after (in #1). There's a browser default 40px margin on a ul. The patch removes it on narrow viewports. Sometimes you need to reload while in the narrow width to see it as it would load narrow.

Got it - it wasn't obvious what to look for even when looking at the screenshots, since the text wasn't really mis-aligned at all, there was just too much whitespace to the left (or right on rtl).

Looks good!

Thanks!

Is the padding really needed for wider screen sizes?

#catch, I figured indented is normal list display, but it will certainly be just as readable either way.

Agreed - the padding could certainly be eliminated for all widths but, in the interest of treading lightly with UX changes, perhaps it shouldn't be...

keyhitman - why are you assigning to yourself? There's a good patch - just a question about whether it should apply for all viewports or only narrow ones. Do you have an opinion on that?

Following discussion at #10 I reworked the patch to apply to all viewports.

Attached screenshots of before and after.

The "More information about text formats" link was also displaying weirdly in the mobile screenshot in the initial summary. Can I get a screenshot for narrow viewports after the patch?

Fixed the "More Information about text formats" link as per #15. Screenshots are attached

+++ b/core/modules/filter/filter.admin.css
@@ -32,7 +32,7 @@
-  padding: 0 20px;
+  padding: 0 20px 0 0;

This is exactly the change we need to make. But it adds a left-to-right specific styling. So you need to add a /* LTR */ comment after the declaration and then add the -rtl version to filter.admin-rtl.css.

Have added -rtl version styling and a /* LTR */ comment as per #17.

Drupal.org is pissing me off. I'm uploading the image from the initial posting again, so I can embed it.

[edit: whoops. didn't mean to RTBC it yet.]

Ok. I've updated the issue summary because I've confirmed that the patch uses the proper solution.

The patch is still in the testing bot queue, but it is a CSS-only fix, so it can't possibly affect any tests. This is RTBC and safe to commit. :-)

Wow, that looks much better! RTL support too! :D

Committed and pushed to 8.x. Thanks!