When changing the link color for dark and black theme we should also change the color of the external link icon. To match development for other icons we should move to using SVGs instead of PNGs.
Description
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Move external link icon from base CSS to PCS CSS | mediawiki/services/mobileapps | master | +30 -22 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T242175 mobile-html: optimize accessibility | |||
| Resolved | • bearND | T251205 Change external link icon to SVG |
Event Timeline
Change 593611 had a related patch set uploaded (by BearND; owner: BearND):
[mediawiki/services/mobileapps@master] Move external link icon from base CSS to PCS CSS
Hi @cmadeo. Is this from https://design.wikimedia.org/style-guide/resources/WikimediaUI-icons-SVGs.zip or a different one?
Hi @bearND it's from a different repository but should be the same, sorry! Feel free to use the one from https://design.wikimedia.org/style-guide/resources/WikimediaUI-icons-SVGs.zip
@cmadeo I think there might be a slight margin difference. Other than that it looks the same. I've updated the apps-ui instance with the icon change to the SVG from the style guide.
- Default: (no fill color)
- Sepia: (fill color: #222)
- Dark: (fill color: #c8ccd1)
- Black: (fill color: #c8ccd1)
Or should the last to be #69f?
Hi @bearND sorry to be so late with this response! On web we style the External link icon as the same color as the active link color, could we do the same here? (see the info box for this article as an example: https://en.wikipedia.org/wiki/Wikipedia)
@cmadeo I've update the fill colors, see the links below. This just covers the base state not the visited state. It's tricky to have many permutations of fill color with the background-image URLs in plain CSS. Once we switch to LESS this could be easier todo. It should be easier now with SVGs vs PNGs, though.
@bearND thanks so much! This looks great. It looks like even on web we maintain the active color for the external link icon even when the link color is visited.
Change 593611 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Move external link icon from base CSS to PCS CSS







