Styling for Views UI rearrange filters

+++ b/core/modules/views_ui/js/views-admin.js
@@ -797,7 +797,7 @@
-        dropdowns = dropdowns.add(fakeOperator);
+        dropdowns.add(fakeOperator);
       }

Shoudn't this issue only focus on the CSS part. The related issue is already about the duplicated/unused variable.

It seems I changed the status before @HOG described how to test it

@HOG or someone else
Do you mind posting a screenshot of before and after?

Before patch

After patch

Thank you! Well I would like to RTBC it, but I cannot judge CSS :(

I'm not good at CSS, but it seems the patch is not following the coding standards

@HOG, the code below is not work for IE browsers properly

.views-ui-rearrange-filter-form table:before {
clear: both;
content: ".";
display: block;
height: 0;
visibility: hidden;
}

I removed these styles and added margin-bottom: 1em to elements above table tag. For now styles works the same like with the patch of @HOG, but works and for IE. Check screen

Needs review

+++ b/core/themes/seven/css/components/views-ui.css
@@ -236,10 +236,16 @@ details.fieldset-no-legend {
   padding: 0;
 }
 
+.views-ui-rearrange-filter-form .tabledrag-toggle-weight-wrapper {
+  float: right;
+  margin-bottom: 1em;
+}
+
 .views-ui-rearrange-filter-form table {

The formatting guidelines states

In general, do NOT separate each ruleset by a blank line. If two rulesets have no interleaving blank line, they must be logically related. If they are not logically related to each other, add a blank line and a comment describing the second ruleset.

I just removed the blank lines in #13. It seems this file has many unnecessary blank lines between rulesets which doesn't happen in other css files. Maybe we need to create another issue to remove them.

thanks for the patch:

Steps used to test:
1. Fresh installed the drupal 8 site.
2. created a view.
3. checked set the filtered criteria and rearranged
4. Create new filter group and Show row weights was not coming in one line.

Applied the patch #15
checked in chrome.
safari and firefox coming fine.
checked in different responsive view looking good.

Remianing IE browser testing else looks good.

@pradeep22saini Is this ready to be marked "Reviewed and tested by the community" status?

Yes, it's working fine in IE also.

Since this is a visual change and not functionally broken, committing to 8.2.x only. I manually tested and it looks great. Thanks all :)

Note: I made a minor whitespace change on commit to try to preserve the intent of the linebreaks grouping the CSS rules, the patch was deleting a linebreak that I think was intended to group/separate things.