Repository: PathOfBuildingCommunity/PathOfBuilding
File: src/Modules/Build.lua (Lua)

Code before:
				self.spec:AddUndoState()
				self.spec:SetWindowTitleWithBuildClass()
				self.buildFlag = true
			else
				main:OpenConfirmPopup("Class Change", "Changing class to "..value.label.." will reset your passive tree.\nThis can be avoided by connecting one of the "..value.label.." starting nodes to your tree.", "Continue", function()
					self.spec:SelectClass(value.classId)
					self.spec:AddUndoState()
					self.spec:SetWindowTitleWithBuildClass()
					self.buildFlag = true					
				end)
			end
		end
	end)
	self.controls.ascendDrop = new("DropDownControl", {"LEFT",self.controls.classDrop,"RIGHT"}, 8, 0, 120, 20, nil, function(index, value)
		self.spec:SelectAscendClass(value.ascendClassId)
		self.spec:AddUndoState()
		self.spec:SetWindowTitleWithBuildClass()
		self.buildFlag = true
	end)
	--self.controls.secondaryAscendDrop = new("DropDownControl", {"LEFT",self.controls.ascendDrop,"RIGHT"}, 8, 0, 120, 20, nil, function(index, value)
	--	self.spec:SelectSecondaryAscendClass(value.ascendClassId)
	--	self.spec:AddUndoState()
	--	self.spec:SetWindowTitleWithBuildClass()
	--	self.buildFlag = true
	--end)

	self.controls.buildLoadouts = new("DropDownControl", {"LEFT",self.controls.ascendDrop,"RIGHT"}, 8, 0, 190, 20, {}, function(index, value)
		if value == "Loadouts:" or value == "-----" then
			self.controls.buildLoadouts:SetSel(1)
			return
		end

		if value == "Sync" then
			self:SyncLoadouts()
			self.controls.buildLoadouts:SetSel(1)
			return
		end

		if value == "New Loadout" then
			local controls = { }
			controls.label = new("LabelControl", nil, 0, 20, 0, 16, "^7Enter name for this loadout:")
			controls.edit = new("EditControl", nil, 0, 40, 350, 20, "New Loadout", nil, nil, 100, function(buf)
				controls.save.enabled = buf:match("%S")
			end)
			controls.save = new("ButtonControl", nil, -45, 70, 80, 20, "Save", function()
				local loadout = controls.edit.buf

				local newSpec = new("PassiveSpec", self, latestTreeVersion)
				newSpec.title = loadout
				t_insert(self.treeTab.specList, newSpec)


Review comment (suggestion):
I'd say add a comment here and in the other place related to `secondaryAscendancy` explaining why is this commented out. May not be obvious to someone trying to contribute a few leagues in the future.

Code after:
				self.spec:AddUndoState()
				self.spec:SetWindowTitleWithBuildClass()
				self.buildFlag = true
			else
				main:OpenConfirmPopup("Class Change", "Changing class to "..value.label.." will reset your passive tree.\nThis can be avoided by connecting one of the "..value.label.." starting nodes to your tree.", "Continue", function()
					self.spec:SelectClass(value.classId)
					self.spec:AddUndoState()
					self.spec:SetWindowTitleWithBuildClass()
					self.buildFlag = true					
				end)
			end
		end
	end)
	self.controls.ascendDrop = new("DropDownControl", {"LEFT",self.controls.classDrop,"RIGHT"}, 8, 0, 120, 20, nil, function(index, value)
		self.spec:SelectAscendClass(value.ascendClassId)
		self.spec:AddUndoState()
		self.spec:SetWindowTitleWithBuildClass()
		self.buildFlag = true
	end)
	-- // hiding away until we learn more, this dropdown and the Loadout dropdown conflict for UI space, will need to address if secondaryAscendancies come back
	--self.controls.secondaryAscendDrop = new("DropDownControl", {"LEFT",self.controls.ascendDrop,"RIGHT"}, 8, 0, 120, 20, nil, function(index, value)
	--	self.spec:SelectSecondaryAscendClass(value.ascendClassId)
	--	self.spec:AddUndoState()
	--	self.spec:SetWindowTitleWithBuildClass()
	--	self.buildFlag = true
	--end)
	self.controls.buildLoadouts = new("DropDownControl", {"LEFT",self.controls.ascendDrop,"RIGHT"}, 8, 0, 190, 20, {}, function(index, value)
		if value == "Loadouts:" or value == "-----" then
			self.controls.buildLoadouts:SetSel(1)
			return
		end
		if value == "Sync" then
			self:SyncLoadouts()
			self.controls.buildLoadouts:SetSel(1)
			return
		end
		if value == "^7Help >>" then
			main:OpenAboutPopup(7)
			self.controls.buildLoadouts:SetSel(1)
			return
		end
		if value == "New Loadout" then
			local controls = { }
			controls.label = new("LabelControl", nil, 0, 20, 0, 16, "^7Enter name for this loadout:")
			controls.edit = new("EditControl", nil, 0, 40, 350, 20, "New Loadout", nil, nil, 100, function(buf)
				controls.save.enabled = buf:match("%S")
			end)
			controls.save = new("ButtonControl", nil, -45, 70, 80, 20, "Save", function()
				local loadout = controls.edit.buf

				local newSpec = new("PassiveSpec", self, latestTreeVersion)