Skip to content
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

Enhancement - Use short URL and copy product link. #24

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

dev-vip
Copy link
Contributor

@dev-vip dev-vip commented Sep 2, 2021

Added get_shortlink in favour of full url get_permalink.
Added share product link button, (needs styling) , perfect for short domain url

tested up to WP 5.8 and Woo 5.6 on live site

Fixes # Links open in new tab, #11
suggested to close.

Changed the way url is displayed, better user experience, especially for short domains.

Added copy product link.

Requires some css changes.

Screenshots

How to test the changes in this Pull Request:

Changelog

Add suggested changelog entry here.

Tweak get_shortlink in favour of full url get_permalink.
Tweak copy product link button, perfect for short domain url
Tweak New copy product link needs styling to your taste. (advised to copy css file to theme child)

Tests

Added get_shortlink in favour of full url get_permalink.
Added share product link button, (needs styling) , perfect for short domain url

tested upto Version 5.8
@dev-vip
Copy link
Contributor Author

dev-vip commented Sep 2, 2021

proposal, add styling panel for copy product link as further improve, if anyone knows how. option, colour of button/text.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant