Remove no-gutters class from templates

I have Reviewed merge request !18. Merge request helped to remove no-gutter classes from template and replace it with gx-0. Need RTBC+1

Attaching a patch to use in composer.json

Tested on my Bootstrap 5 based site. Setting no gutters now functions as expected.

I'm not sure if the version configuration item is really needed though? Why not set both classes? The one that matches the current bootstrap version will work, the other won't do anything.

Are there other differences between them that apply to this module, not bootstrap_styles, that would benefit from the version configuration item?

Hi!

I tested patch #10 with version 2.1.2 and all worked as expected.
I'm moving to reviewed and tested.

Guys, this thing is essentially done. Although the comment by @rhovland does make sense if we have a more strict focus on the particular problem covered by this issue queue, the Bootstrap selector makes sense if other class name conflicts arise in the future (requiring us to apply additional conditions based on the framework version).

We should sign, seal, and deliver this thing so we don't need to keep applying this same patch over and over.

rajab natshah → made their first commit to this issue’s fork.

1. Add Schema item for bootstrap_version
2. Needs a hook update to update old settings

The MR has been updated.