feat: add watermark to the advanced submenu#969
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds watermark functionality to the Optimole WordPress plugin's dashboard interface, allowing users to access watermark settings through a new submenu item under Advanced Settings.
- Adds a new "Watermark" menu item to the Advanced Settings submenu
- Creates a dedicated Watermark component with informational content and links to external configuration
- Integrates watermark strings and UI elements into the existing dashboard structure
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| inc/admin.php | Adds localization strings for watermark UI elements and descriptions |
| assets/src/dashboard/parts/connected/settings/index.js | Integrates Watermark component into settings tabs and conditionally hides save button |
| assets/src/dashboard/parts/connected/settings/Watermark.js | Creates new Watermark component with feature information and external links |
| assets/src/dashboard/parts/connected/settings/Menu.js | Adds watermark menu item to Advanced Settings submenu |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| <h1 className="text-xl font-bold"> | ||
| {options_strings.watermark_media_title} | ||
| </h1> | ||
| <p className="text-gray-600 mt-2 mb-2" > |
There was a problem hiding this comment.
Extra space before the closing angle bracket. Remove the space before >.
| </div> | ||
| <div className="flex items-center gap-2 align-center"> | ||
| <div className="w-2 h-2 bg-gray-900 rounded-full flex-shrink-0"></div> | ||
| <span className="text-gray-700 text-left ">{options_strings.watermark_info_2}</span> |
There was a problem hiding this comment.
Extra space in the className attribute before the closing quote. Remove the trailing space after text-left.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
🎉 This PR is included in version 4.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
pirate-bot
added
the
released
All Submissions:
Changes proposed in this Pull Request:
Added exposure in the dashboard for the watermark feature.
Closes https://github.com/Codeinwp/optimole-service/issues/1509
Other information: