Merge pull request #8922 from nextcloud/enh/8896/more-breakout-rooms-polishing

Enh/8896/more breakout rooms polishing

src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue

@@ -38,24 +38,26 @@
 						max="20">
 
 					<label class="breakout-rooms-editor__caption">{{ t('spreed', 'Assignment method') }}</label>
-					<NcCheckboxRadioSwitch :checked.sync="mode"
-						value="1"
-						name="mode_radio"
-						type="radio">
-						{{ t('spreed', 'Automatically assign participants') }}
-					</NcCheckboxRadioSwitch>
-					<NcCheckboxRadioSwitch :checked.sync="mode"
-						value="2"
-						name="mode_radio"
-						type="radio">
-						{{ t('spreed', 'Manually assign participants') }}
-					</NcCheckboxRadioSwitch>
-					<NcCheckboxRadioSwitch :checked.sync="mode"
-						value="3"
-						name="mode_radio"
-						type="radio">
-						{{ t('spreed', 'Allow participants to choose') }}
-					</NcCheckboxRadioSwitch>
+					<fieldset>
+						<NcCheckboxRadioSwitch :checked.sync="mode"
+							value="1"
+							name="mode_radio"
+							type="radio">
+							{{ t('spreed', 'Automatically assign participants') }}
+						</NcCheckboxRadioSwitch>
+						<NcCheckboxRadioSwitch :checked.sync="mode"
+							value="2"
+							name="mode_radio"
+							type="radio">
+							{{ t('spreed', 'Manually assign participants') }}
+						</NcCheckboxRadioSwitch>
+						<NcCheckboxRadioSwitch :checked.sync="mode"
+							value="3"
+							name="mode_radio"
+							type="radio">
+							{{ t('spreed', 'Allow participants to choose') }}
+						</NcCheckboxRadioSwitch>
+					</fieldset>
 				</div>
 				<div class="breakout-rooms-editor__buttons">
 					<NcButton v-if="mode === '2'" type="primary" @click="isEditingParticipants = true">

src/components/BreakoutRoomsEditor/BreakoutRoomsParticipantsEditor.vue

@@ -81,6 +81,7 @@
 				{{ resetButtonLabel }}
 			</NcButton>
 			<NcActions v-if="hasSelected"
+				type="primary"
 				:menu-title="t('spreed', 'Assign')">
 				<NcActionButton v-for="(item, index) in assignments"
 					:key="index"
@@ -93,7 +94,7 @@
 				</NcActionButton>
 			</NcActions>
 			<NcButton :disabled="!hasAssigned"
-				type="primary"
+				:type="confirmButtonType"
 				@click="handleSubmit">
 				{{ confirmButtonLabel }}
 			</NcButton>
@@ -202,6 +203,10 @@ export default {
 			return this.isReorganizingAttendees ? t('spreed', 'Confirm') : t('spreed', 'Create breakout rooms')
 		},
 
+		confirmButtonType() {
+			return this.hasUnassigned ? 'secondary' : 'primary'
+		},
+
 		resetButtonLabel() {
 			return t('spreed', 'Reset')
 		},