Skip to content

✨ Add brand logo and page elements settings - #2933

Open
b-l-i-n-d wants to merge 55 commits into
devfrom
feat/logo
Open

✨ Add brand logo and page elements settings#2933
b-l-i-n-d wants to merge 55 commits into
devfrom
feat/logo

Conversation

@b-l-i-n-d

@b-l-i-n-d b-l-i-n-d commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator
  • Adds dark mode icon to icon set
  • Adds brand logo template with light/dark variant support
  • Adds brand logo upload settings field in Appearance
  • Adds page elements settings to control header/footer visibility per page
  • Replaces Spotlight Mode toggle with granular Page Elements controls, preserving existing Spotlight Mode behavior during migration to Page Elements
  • Removes hardcoded HTML boilerplate from templates in favor of helper functions
  • Supports resetting image_upload_list and toggle_matrix settings fields
  • Migrates the frontend course builder logo to the light brand logo setting

Linked PR: https://github.com/themeum/tutor-pro/pull/1885

@b-l-i-n-d
b-l-i-n-d changed the base branch from master to dev July 29, 2026 09:52
@b-l-i-n-d b-l-i-n-d changed the title ✨ Add brand logo and page elements settings 🧹 Remove unused CSS selectors from brand logo and page elements Jul 29, 2026
@b-l-i-n-d b-l-i-n-d changed the title 🧹 Remove unused CSS selectors from brand logo and page elements 🧹 Rename brand logo CSS classes and change dashboard header/footer defaults Jul 29, 2026
@b-l-i-n-d b-l-i-n-d changed the title 🧹 Rename brand logo CSS classes and change dashboard header/footer defaults ✨ Add brand logo and page elements settings Jul 29, 2026
@b-l-i-n-d b-l-i-n-d self-assigned this Jul 29, 2026
@b-l-i-n-d
b-l-i-n-d marked this pull request as ready for review July 30, 2026 05:24
@b-l-i-n-d
b-l-i-n-d requested a review from shewa12 July 30, 2026 05:24
Comment thread views/options/field-types/brand_logo.php Outdated
Comment thread views/options/field-types/page_elements.php Outdated
- Replace brand-logo specific field with reusable image_upload_list
- Update JS and SCSS selectors to the new generic classes
- Replace page_elements specific field with reusable toggle_matrix
- Update SCSS selectors to the new generic classes
@b-l-i-n-d
b-l-i-n-d requested a review from sazedul-haque August 4, 2026 05:50
@b-l-i-n-d b-l-i-n-d removed the 4.0.4 label Aug 5, 2026
Comment thread assets/src/scss/frontend/dashboard/layout/_nav-mobile.scss
Comment thread assets/src/js/v3/shared/components/Logo.tsx Outdated
Comment thread classes/Options_V2.php

if ( 'imported' === $action ) {
if ( ! empty( $save_import_data ) ) {
if ( ! empty( $save_import_data['dataset'] ) ) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this changes?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This resolved phpstan warnings

Comment thread classes/Options_V2.php

if ( 'imported' === $action ) {
if ( ! empty( $save_import_data ) ) {
if ( ! empty( $save_import_data['dataset'] ) ) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This resolved phpstan warnings

$multiple_select = "multiple='multiple'";
}

$classes = '';

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this changes?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This resolved phpstan warnings

$multiple_select = "multiple='multiple'";
}

$classes = '';

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This resolved phpstan warnings

Comment thread includes/tutor-general-functions.php Outdated
Comment thread templates/dashboard-isolated.php Outdated
Comment thread templates/dashboard.php Outdated
Comment thread templates/dashboard.php Outdated
@@ -0,0 +1,107 @@
<?php

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file name should be - dash instead of underscore

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All fields currently use underscores (_) in their naming convention; using dashes (-) would be an antipattern.

@@ -0,0 +1,101 @@
<?php

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here for file name

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All fields currently use underscores (_) in their naming convention; using dashes (-) would be an antipattern.

Comment thread classes/Course.php Outdated
Comment thread classes/Options_V2.php
@b-l-i-n-d b-l-i-n-d added 4.0.7 Tutor v4.0.7 and removed 4.0.6 Tutor v4.0.6 labels Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.0.7 Tutor v4.0.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants