-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance: Update dokan admin dashboard logo & add logo update notice. #2258
base: develop
Are you sure you want to change the base?
Enhance: Update dokan admin dashboard logo & add logo update notice. #2258
Conversation
@shohag121 Vhaiya, kindly review this PR at your convenience. |
…ription & remove one-step notice.
@mrabbani Vhai, need to relevant ui's for new issues. |
@MdAsifHossainNadim, @mrabbani, |
Here is the Design Link. Please have a look. |
@tuhin480 Vhai, we need to make consistent between existing banner & updated logo. Here is the preview after updating logo. I think It will be better if we added some shadow or spaces on left/bottom side in this banner image. (as like as previous) We need to update wp org's banner & icon logo's as per latest update. |
Enhance: Update dokan admin logo & add logo update notice.
All Submissions:
Changes proposed in this Pull Request:
Related Pull Request(s)
Closes
How to test the changes in this Pull Request:
Changelog entry
Detailed Description of the pull request. What was previous behaviour
and what will be changed in this PR.
Before Changes
After Changes
Feature Video (optional)
Link of detailed video if this PR is for a feature.
PR Self Review Checklist:
FOR PR REVIEWER ONLY: