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/5454 make links to shops under Groups > Producers open in new tab #5601

Conversation

jeduardo824
Copy link
Contributor

@jeduardo824 jeduardo824 commented Jun 15, 2020

What? Why?

Closes #5454

What should we test?

If the shops' links are opening in a new tab properly (both producers name and shops when the tab expands).

Release notes

Make shops' links on groups page open in a new tab (both producer name and shops when the tab expands).
Changelog Category: Changed

Result:
ofn
ofn

@jeduardo824 jeduardo824 changed the title Enhancement/5454 make links to shops under Groups > Producers open in new tab WIP Enhancement/5454 make links to shops under Groups > Producers open in new tab Jun 15, 2020
@jeduardo824
Copy link
Contributor Author

@tschumilas Requested changes! Please, do not merge!

Copy link
Contributor

@Matt-Yorkley Matt-Yorkley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jeduardo824 jeduardo824 force-pushed the enhancement/5454-links-to-shops-in-new-tabs branch from c537677 to 573f7b1 Compare June 18, 2020 22:55
@jeduardo824 jeduardo824 changed the title WIP Enhancement/5454 make links to shops under Groups > Producers open in new tab Enhancement/5454 make links to shops under Groups > Producers open in new tab Jun 18, 2020
@jeduardo824
Copy link
Contributor Author

Ready to go!!

@filipefurtad0 filipefurtad0 self-assigned this Jul 2, 2020
@filipefurtad0 filipefurtad0 added the pr-staged-fr staging.coopcircuits.fr label Jul 2, 2020
@filipefurtad0
Copy link
Contributor

Hey @jeduardo824 ,

Nice! These links now open in separate tabs - Thank you for this improvement.
Good to go.

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

Successfully merging this pull request may close these issues.

Can the links to shops on the group page open into new tabs?
5 participants