Reverted light sizes in components.hpp, updated README

README.md

@@ -13,6 +13,8 @@ However, please search before posting to avoid duplicates, and limit to one issu
 
 You may vote on feature requests by using the Thumbs Up/Down reaction on the first post.
 
+I rarely accept Pull Requests, so please notify me in advance to plan your contribution before writing code.
+
 ## Setting up your development environment
 
 Rack's dependencies (GLEW, glfw, etc) do not need to be installed on your system, since specific versions are compiled locally during the build process. However, you need proper tools to build these dependencies.
@@ -63,7 +65,7 @@ Run Rack.
 
 Be sure to check out and build the version of Rack you wish to build your plugins against.
 
-Clone your favorite plugin in the `plugins/` directory. e.g.:
+You must clone the plugin in Rack's `plugins/` directory, e.g.
 
 	cd plugins
 	git clone https://github.com/VCVRack/Fundamental.git

include/components.hpp

@@ -434,6 +434,19 @@ struct GreenLight : ColorLightWidget {
 	}
 };
 
+struct YellowLight : ColorLightWidget {
+	YellowLight() {
+		addColor(COLOR_YELLOW);
+	}
+};
+
+struct BlueLight : ColorLightWidget {
+	BlueLight() {
+		addColor(COLOR_BLUE);
+	}
+};
+
+/** Reads two adjacent lightIds, so `lightId` and `lightId + 1` must be defined */
 struct GreenRedLight : ColorLightWidget {
 	GreenRedLight() {
 		addColor(COLOR_GREEN);
@@ -446,7 +459,7 @@ struct GreenRedLight : ColorLightWidget {
 template <typename BASE>
 struct LargeLight : BASE {
 	LargeLight() {
-		this->box.size = Vec(15, 15);
+		this->box.size = Vec(20, 20);
 	}
 };
 
@@ -454,7 +467,7 @@ struct LargeLight : BASE {
 template <typename BASE>
 struct MediumLight : BASE {
 	MediumLight() {
-		this->box.size = Vec(9, 9);
+		this->box.size = Vec(12, 12);
 	}
 };
 
@@ -462,7 +475,7 @@ struct MediumLight : BASE {
 template <typename BASE>
 struct SmallLight : BASE {
 	SmallLight() {
-		this->box.size = Vec(6, 6);
+		this->box.size = Vec(8, 8);
 	}
 };
 
@@ -470,7 +483,7 @@ struct SmallLight : BASE {
 template <typename BASE>
 struct TinyLight : BASE {
 	TinyLight() {
-		this->box.size = Vec(3, 3);
+		this->box.size = Vec(5, 5);
 	}
 };
 

src/app/ColorLightWidget.cpp

@@ -10,6 +10,7 @@ void ColorLightWidget::addColor(NVGcolor c) {
 }
 
 void ColorLightWidget::step() {
+	assert(module);
 	color = nvgRGBf(0, 0, 0);
 	for (int i = 0; i < (int)colors.size(); i++) {
 		NVGcolor c = colors[i];