0
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2026-08-04 10:39:43 +08:00
discourse/app/assets/stylesheets/common
Kris 665494fc53
DEV: simplify and use reuable components in invite modal, fix bug, (#42273)
We were creating some new conventions here instead of relying on
existing components. This gets the invite modal on more shared
components, reduces the custom css, and fixes the bug reported here:
https://meta.discourse.org/t/invite-restricted-cant-remove/409058

reusing existing components
- segment control 
- formkit conditional section
- advanced mode toggle  


Removed some redundant text: 
"invite people to join this site" — what else would you be inviting
people to?
"share a link or send an email invitation" – that's what the radio
options already indicate
"to invite a specific person, use email above" – that's implied by
"email"

before
<img width="600" alt="image"
src="https://github.com/user-attachments/assets/1cdb474c-e393-4066-854f-3b6d714b0195"
/>

after
<img width="600" alt="image"
src="https://github.com/user-attachments/assets/506229b7-88fe-437f-8a2d-db3618814c48"
/>


I've also triggered the success button on the copy button on the second
step, as an indication that the link has been copied successfully.

<img width="600" alt="image"
src="https://github.com/user-attachments/assets/52c9ef8d-94ea-4b11-923d-9e5745a7e53e"
/>


Changed the order here so the descriptions are associated with the
selection and below them (rather than being above them, which is awkward
for reading order)

Removed "invite people to join your staff team" which was redundant, the
roles describe what kind of access you're inviting — also elaborated a
little to make the full admin scope more apparent

before
<img width="600" alt="image"
src="https://github.com/user-attachments/assets/9f4684ec-b2d0-43d6-92b7-23dd27156f77"
/>


after
<img width="600" alt="image"
src="https://github.com/user-attachments/assets/b7af2317-21c4-4b0c-809a-c4ac13c218e3"
/>


Mobile also fits slightly better on the tiniest screens at 320px wide 

before
<img width="320" alt="image"
src="https://github.com/user-attachments/assets/11e61205-f3a6-4f68-a8ea-0c1ef75b9aa1"
/>

after
<img width="320" alt="image"
src="https://github.com/user-attachments/assets/b2caef7d-5562-452f-8d51-6c9af779a809"
/>
2026-08-03 15:31:55 -04:00
..
base DEV: simplify and use reuable components in invite modal, fix bug, (#42273) 2026-08-03 15:31:55 -04:00
components DEV: simplify and use reuable components in invite modal, fix bug, (#42273) 2026-08-03 15:31:55 -04:00
float-kit
form-kit DEV: simplify and use reuable components in invite modal, fix bug, (#42273) 2026-08-03 15:31:55 -04:00
foundation UX: add 2 border-radius vars (#41346) 2026-07-01 22:17:40 +02:00
login UX: Improve password manager and full name handling in code login (#41746) 2026-07-15 10:43:03 -07:00
modal DEV: simplify and use reuable components in invite modal, fix bug, (#42273) 2026-08-03 15:31:55 -04:00
rich-editor
select-kit UX: Disable ACL grantee row when all permissions banned (#41865) 2026-07-23 10:44:42 +10:00
table-builder
upcoming-changes UX: UC: composer redesign > prevent accidental clicks to close expanded editor (#42160) 2026-07-30 14:23:52 +02:00
d-editor.scss UX: Indicate when the emoji picker category list is scrollable (#41858) 2026-07-21 11:18:35 -03:00
font-variables.scss
input_tip.scss
loading-slider.scss
nested-view.scss FIX: Add small action controls to nested activity log (#42108) 2026-07-29 10:19:47 -05:00
post-action-feedback.scss
printer-friendly.scss
software-update-prompt.scss
tokens.scss
toolbar-popup-menu-options.scss FEATURE: Add a small text option to the composer text sizes toolbar button (#42020) 2026-08-03 11:28:59 +02:00
topic-entrance.scss
topic-timeline.scss
whcm.scss