refactor: UI for Schedule option in Settings#1141
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the import scheduler interface by changing the label text and removing the help description. The changes focus on improving the visual presentation of the import schedule configuration.
- Updated the label from "Import schedule" to "Default Importing Schedule"
- Removed the help text that explained the scheduler functionality
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| <div class="fz-form-group"> | ||
| <label class="form-label"><?php esc_html_e( 'Import schedule', 'feedzy-rss-feeds' ); ?></label> | ||
| <label class="form-label"><?php esc_html_e( 'Default Importing Schedule', 'feedzy-rss-feeds' ); ?></label> | ||
| <select id="fz-event-schedule" class="form-control fz-select-control" name="feedzy_meta_data[fz_cron_schedule]"<?php disabled( true, ! feedzy_is_pro() ); ?>> |
There was a problem hiding this comment.
Removing the help text that explained 'Choose how often Feedzy should import new items from your feeds' may reduce accessibility for users who need context about what the schedule setting does. Consider adding aria-describedby or title attributes to maintain accessibility.
| <select id="fz-event-schedule" class="form-control fz-select-control" name="feedzy_meta_data[fz_cron_schedule]"<?php disabled( true, ! feedzy_is_pro() ); ?>> | |
| <label class="form-label" for="fz-event-schedule"><?php esc_html_e( 'Default Importing Schedule', 'feedzy-rss-feeds' ); ?></label> | |
| <div id="fz-schedule-help" class="screen-reader-text"> | |
| <?php esc_html_e( 'Choose how often Feedzy should import new items from your feeds.', 'feedzy-rss-feeds' ); ?> | |
| </div> | |
| <select id="fz-event-schedule" class="form-control fz-select-control" name="feedzy_meta_data[fz_cron_schedule]" aria-describedby="fz-schedule-help"<?php disabled( true, ! feedzy_is_pro() ); ?>> |
|
Plugin build for 183d31f is ready 🛎️!
Note You can preview the changes in the Playground |
|
@Soare-Robert-Daniel, The changes are reflected correctly in the general settings, but shouldn’t they appear the same way in the individual import as well? The per-import schedule setting isn’t displaying "externally created" text with the manually created cron schedule. |
dcd04f8 to
183d31f
Compare
|
🎉 This PR is included in version 5.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |

Summary
Removed description and changed the title.
Will affect visual aspect of the product
YES
Screenshots
Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/914