Page MenuHomePhabricator

[Share Highlights] Broken PNG download when the share card's background is dark or transparent
Closed, ResolvedPublic3 Estimated Story PointsBUG REPORT

Description

When the dark background is selected, the downloaded PNG preserves the font color, but the background becomes white, making the quoted text invisible.
Share quote dialog example:

Screenshot 2026-03-25 at 12.58.28.png (1×770 px, 774 KB)

Downloaded PNG:
wikipedia-quote-robert-crumb.png (1×624 px, 646 KB)

Update

When the transparent background is selected, the downloaded PNG doesn't seem to preserve the transparency.
Share quote dialog example:
{F76371302}

Downloaded PNG:

wikipedia-quote-marna--fiume-(2).png (1×640 px, 884 KB)

NOTE: in light of the update, we may need to re-point the task.

Event Timeline

HSwan-WMF set the point value for this task to 3.Mar 24 2026, 4:37 PM
mfossati renamed this task from [Share Highlights] Broken PNG download when the background is dark to [Share Highlights] Broken PNG download when the share card's background is dark or transparent.Fri, Apr 17, 10:11 AM
mfossati updated the task description. (Show Details)

Change #1273754 had a related patch set uploaded (by LWatson; author: LWatson):

[mediawiki/extensions/ReaderExperiments@master] Share Highlights: fix PNG download

https://gerrit.wikimedia.org/r/1273754

Test wiki on Patch demo by LWatson-WMF using patch(es) linked to this task was deleted:

https://d706b87645.catalyst.wmcloud.org/w/

Test wiki on Patch demo by LWatson-WMF using patch(es) linked to this task was deleted:

https://fb5eb727ec.catalyst.wmcloud.org/w/

✅ Tested downloaded PNG on Firefox browser.
✅ Tested the share card via AirDrop and Apple Notes.

Notes:

  • Safari's native share sheet displays a PNG thumbnail placeholder.
  • The transparent background can appear broken on devices with dark themes. I experienced this when viewing the share card in Apple's Preview, Notes, and Photos apps. The example below is a screenshot from Apple's Notes app on macOS with dark appearance in system settings. Similar issue with Apple Photos.
    ShareHighlights - transparent card in Notes.png (1×1 px, 1 MB)
    • For Preview, you can adjust the window background in Settings via Preview > Settings > Window background
      ShareHighlight - transparent card in Preview with dark window background.png (1×2 px, 2 MB)
      ShareHighlight - transparent card in Preview with purple window background.png (1×1 px, 2 MB)
  • Based on findings, do you think we should remove the transparent background option or keep it as is? Filed a follow-up: T423734: Share Highlight: transparent background card has invisible text in dark themed apps

cc: @JScherer-WMF

egardner changed the task status from Open to In Progress.Mon, Apr 20, 7:14 PM

Moving back to code review after considering Minerva's night/light modes T423688: [Share Highlights] Light and transparent backgrounds don't display properly in Minerva's dark theme.

Update: In the dialog preview, the dark and light background color options use fixed font and background colors; therefore, the card is consistent between Minerva's night/light mode. Using a fixed font color for the transparent option makes the text invisible over a dark background in the dialog preview; therefore, the transparent option's text color remains responsive to modes. For example, when Minerva's night mode is enabled, the transparent option's font color is white, and when light mode is enabled, the transparent card's font color is black.

Change #1273754 merged by jenkins-bot:

[mediawiki/extensions/ReaderExperiments@master] Share Highlights: fix PNG download

https://gerrit.wikimedia.org/r/1273754

Moved back to "Doing" - next steps per Justin in Slack:

layer a white background behind the transparent card in the preview so that the text colour doesn't change

Change #1276795 had a related patch set uploaded (by LWatson; author: LWatson):

[mediawiki/extensions/ReaderExperiments@master] Share Highlight: remove transparent background color option

https://gerrit.wikimedia.org/r/1276795

Moving to code review for change 1276795. Team discussed removing the transparent option for now and resuming it if time permits before the experiment launch date.

Change #1276795 merged by jenkins-bot:

[mediawiki/extensions/ReaderExperiments@master] Share Highlight: remove transparent background color option

https://gerrit.wikimedia.org/r/1276795

lwatson changed the subtype of this task from "Task" to "Bug Report".EditedSat, Apr 25, 1:09 PM

Moving to sign off and skipping QA because we decided to remove background color customization T424424: Share Highlight: remove card customization (background color options). The experiment will ship the average background color. Transparent option removal found in this change: https://gerrit.wikimedia.org/r/1276795