Open
Conversation
Introduce a new StatusSelect UI component: adds PHP builder (components/StatusSelect.php) for server rendering and Alpine props, a TypeScript Alpine controller (assets/core/ts/components/status-select.ts) to handle variant classes and AJAX status updates, and SCSS styling with variant tokens (assets/core/scss/components/_status-select.scss). Also export the component in the core SCSS index and register statusSelectMeta in the TS index so it initializes with other components.
Introduce a new $grid-split-utils SCSS map with predefined non-uniform column splits and a loop to generate .tutor-grid-cols-<name> classes (e.g. '1-5-1': 1.5fr 1fr, '2-1': 2fr 1fr, '1-2': 1fr 2fr). This provides specific grid-template-columns options in addition to the existing repeat-based grid column utilities and preserves .tutor-grid-cols-none.
Introduce a new protected property $menu_min_width and a fluent setter menu_min_width(string): self. When set, the popover menu wrapper will include an inline style to apply a min-width. Includes docblocks and an @SInCE tag for the new API.
harunollyo
requested changes
Mar 3, 2026
|
|
||
| use TUTOR\Icon; | ||
|
|
||
| defined( 'ABSPATH' ) || exit; |
Collaborator
There was a problem hiding this comment.
Use this immediate after namespace.
| > | ||
| <?php foreach ( $this->options as $value => $option ) : ?> | ||
| <option value="<?php echo esc_attr( $value ); ?>" <?php selected( $this->selected, $value ); ?>> | ||
| <?php echo esc_html( $option['label'] ); ?> |
| 'selected' => $this->selected, | ||
| 'action' => $this->action, | ||
| 'data' => $this->data, | ||
| 'variants' => array_map( |
Collaborator
There was a problem hiding this comment.
single statement. use arrow fn
'variants' => array_map( fn( $opt ) => $opt['variant'] ?? 'default', $options )
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Status component added to change status from list.
