General RGB matrix fixes (#2931)

* Added Modular keyboards L,R and NUM

Created code modules for the 3 modules of the modular keyboard.
Original idea by MechboardsUK. Uses i2c implementation similar to lets
split

* Remove modular from master

This is to fix incorrect branching

* General fixes for RGB_matrix

- Complited speed support for all effects
- Fixed raindrop effects to initialized after toggle
- Fixed raindrop effects to use all available LEDs
- Fixed effect step reverse function
- Moved RGB_MATRIX_SOLID_REACTIVE under correct flag

* Documentation update for RGBmatrix

* More doc updates

1 files changed, 2 insertions, 2 deletions

diff --git a/docs/feature_rgb_matrix.md b/docs/feature_rgb_matrix.md
index 084e87ec4..5d2db3b97 100644
--- a/docs/feature_rgb_matrix.md
+++ b/docs/feature_rgb_matrix.md
@@ -81,7 +81,6 @@ These are the effects that are currently available:
 
         enum rgb_matrix_effects {
                 RGB_MATRIX_SOLID_COLOR = 1,
-            RGB_MATRIX_SOLID_REACTIVE,
             RGB_MATRIX_ALPHAS_MODS,
             RGB_MATRIX_DUAL_BEACON,
             RGB_MATRIX_GRADIENT_UP_DOWN,
@@ -94,6 +93,7 @@ These are the effects that are currently available:
             RGB_MATRIX_RAINBOW_MOVING_CHEVRON,
             RGB_MATRIX_JELLYBEAN_RAINDROPS,
         #ifdef RGB_MATRIX_KEYPRESSES
+                RGB_MATRIX_SOLID_REACTIVE,
             RGB_MATRIX_SPLASH,
             RGB_MATRIX_MULTISPLASH,
             RGB_MATRIX_SOLID_SPLASH,
@@ -118,7 +118,7 @@ A similar function works in the keymap as `rgb_matrix_indicators_user`.
         #define RGB_MATRIX_KEYRELEASES // reacts to keyreleases (not recommened)
         #define RGB_DISABLE_AFTER_TIMEOUT 0 // number of ticks to wait until disabling effects
         #define RGB_DISABLE_WHEN_USB_SUSPENDED false // turn off effects when suspended
-    #define RGB_MATRIX_SKIP_FRAMES 1 // number of frames to skip when displaying animations (0 is full effect)
+    #define RGB_MATRIX_SKIP_FRAMES 1 // number of frames to skip when displaying animations (0 is full effect) if not defined defaults to 1
 
 ## EEPROM storage