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

PLANET-4334 Allow a list of non-P4 urls not to have the ↗️ icon #174

Open
planet-4 opened this issue Sep 30, 2021 · 7 comments
Open
Assignees
Labels
Dev development tickets Feature Moderate

Comments

@planet-4
Copy link

planet-4 commented Sep 30, 2021

We add an external icon on links that point to an external domain. As a follow up we should not display the external link icon for some links we use regularly for engagement opportunities (eg. act.greenpeace.org).

Requirements

  • Add a new field "Commonly used external domains" under Planet 4 > Social.
  • Use a textaea field to hold multiple domains.
  • Help text: "Links for these domains will not open in a new tab and will hide the external link icon."
  • Both title and description should be translatable.
  • Adjust external link behaviour to remove external link icon and opening in a new tab for these domains.
Reporter: ltiralon
Sections: Styling/Theme
@planet-4 planet-4 added Dev development tickets Feature Moderate labels Sep 30, 2021
@oekeur
Copy link

oekeur commented Jan 24, 2022

Hi friends 🤗

Sure this won't interfere in a weird way if it's implemented like this?
The pre-connect domains are a technical solution for pre-connecting certain assets. If we piggyback on this by refactoring it to "Preferred external domains", that would mean that certain third party domains unintentionally might also lack the external link icon..
I know it's a bit of a edge case, but just to be sure. :)

@comzeradd
Copy link
Member

Yeah, we discussed this also in the team. It may make more sense to have an explicit configuration option for this. I'll update the description requirements.

@oekeur
Copy link

oekeur commented Apr 7, 2022

So what shall we do for this? :)
A separate option for the 'domains without link icon'?

@comzeradd
Copy link
Member

Description updated. Let's add a new configuration option for this.

@oekeur
Copy link

oekeur commented Apr 12, 2022

You can assign this one to me :)

@suzi-greenpeace
Copy link
Contributor

suzi-greenpeace commented Apr 12, 2022 via email

@oekeur
Copy link

oekeur commented May 23, 2022

Whoops, sorry, this one fell off my radar!:
Starting -now really- on it now

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

No branches or pull requests

4 participants