Aligning Apply button in Views Exposed Filter

ravi kant → made their first commit to this issue’s fork.

@egfrith

The requirement can archive using style in your custom theme.
But I have made changes in base theme and created MR.

Thank you for looking at this and producing the suggestion ravi. I've taken a look at the code. In Firefox, I'm not sure it worked - though I was having all sorts of problems with CSS being cached. I'm also wondering if the hard coding of the height of the Apply button via the margin-tope might be a bit fragile.

I've looked again at the way views does the exposed filters. The code there uses flexes, and the SASS code below works OK in my subtheme:

form--inline {
  display: flex;
  .form-actions {
    display: flex;
    flex-wrap: wrap;
    align-items: flex-end;
  }
}

I'm not very expert on CSS and bootstrap, so I'm not sure if this is the optimal solution - I would defer to the experts!

@egfrith
Your style code is perfect for sub theme but base theme not using sass for form--inline style so i made changes directly in style file.

See @ravi kant suggestion and update the fix.
Cheers

I have applied styles on SASS files.

@ravi kant thanks for the patch - to speed up review:
since you've already created a patch and setup the environment - can you please provide screenshots of before/after patch application from the browsers: safari/firefox/chrome/edge

@jannakha
Resolved conflict and attaching screenshots of "before and after" apply changes.

I have tested the Merge Request in #3 and tried to applied it as a patch but it throwed some errors. Following that, pulled the branch and then tested it and it is working as good and it solves the issue. Attaching screenshots for reference.
Thanks and regards,

@Hritick the patch couldn't be applied because the branch was really old.
I have updated this fork to the latest 3.0.x - try to apply patch now?

Hey @jannakha,
Tried to reapply the patch but still it failed. Using curl for applying the patch.

thanks and regards.