Allow specific roles to configure custom CSS on the site. Usually, specific roles on the site manage the styling, so it will be good if we add permission and a check while displaying custom CSS theme settings. adding patch for the same.

CommentFileSizeAuthor
#11 3060624-11.patch1016 byteschaitanyadessai
#8 permission-for-css-theme-settings-3060624-8.patch1001 byteschintan.vyas
#6 css_editor-permission-for-css-theme-settings-3060624-6.patch1.38 KBchintan.vyas
#5 permission-for-css-theme-settings-3060624-5.patch993 byteschintan.vyas
#2 permission-for-css-theme-settings-3060624-1.patch979 byteschintan.vyas

Comments

chintan.vyas created an issue. See original summary.

chintan.vyas’s picture

chintan.vyas commented
StatusFileSize
new permission-for-css-theme-settings-3060624-1.patch979 bytes
chintan.vyas’s picture

chintan.vyas commented
Title: Adding permissions for custom CSS theme settings » Adding permission for custom CSS theme settings
chintan.vyas’s picture

chintan.vyas commented
chintan.vyas’s picture

chintan.vyas commented
StatusFileSize
new permission-for-css-theme-settings-3060624-5.patch993 bytes

Updated patch

chintan.vyas’s picture

chintan.vyas commented
StatusFileSize
new css_editor-permission-for-css-theme-settings-3060624-6.patch1.38 KB

Updated patch.

chintan.vyas’s picture

chintan.vyas commented
Status: Active » Needs review
chintan.vyas’s picture

chintan.vyas commented
StatusFileSize
new permission-for-css-theme-settings-3060624-8.patch1001 bytes

Sorry for the confusion regarding the tag 8.x-1.2. This patch is against the current tag 8.x-1.2.

bajah1701’s picture

bajah1701 commented

I would also recommend to add

restrict access: TRUE

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs review » Needs work

Needs Work for #9.

chaitanyadessai’s picture

chaitanyadessai commented
Status: Needs work » Needs review
StatusFileSize
new 3060624-11.patch1016 bytes

#9 changes addressed Please review.