backdropcms.org icon indicating copy to clipboard operation
backdropcms.org copied to clipboard

Download icon on project page replaced with empty square in new design

Open bugfolder opened this issue 6 months ago • 11 comments

Here's what a download used to look like on a project page (e.g., https://backdropcms.org/project/ubercart):

Image

Here's what it looks like now:

Image

The empty square is a bit puzzling (perhaps unintended?).

Using Safari on Mac, OS X Sequoia. It's also this way in Chrome. Firefox shows something different, but still not as good as a down-arrow:

Image

bugfolder avatar Jun 23 '25 21:06 bugfolder

I can confirm this. I noticed it yesterday while exploring the new site.

argiepiano avatar Jun 23 '25 23:06 argiepiano

It's because that icon is a FontAwesome character. In the old theme, FontAwesome was included in function borg_preprocess_page():

  // Add FontAwesome.
  backdrop_add_js('https://use.fontawesome.com/baf3c35582.js', array('type' => 'external'));

But in the new theme, that line is commented out and something called "ForkAwesome" is added:

  // Add FontAwesome.
  //$font_awesome = 'https://use.fontawesome.com/baf3c35582.js';
  //backdrop_add_js($font_awesome, array('type' => 'external'));

  // Add ForkAwesome. @todo - replace with ICON API
  $fork_awesome = 'https://cdn.jsdelivr.net/npm/[email protected]/css/fork-awesome.min.css';
  $attributes = array(
    'integrity' => 'sha256-XoaMnoYC5TH6/+ihMEnospgm0J1PM/nioxbOUdnM8HY=',
    'crossorigin' => 'anonymous',
  );
  backdrop_add_css($fork_awesome, array('type' => 'external', 'attributes' => $attributes));

As the TODO says, perhaps it would be better to use core icons anyhow.

bugfolder avatar Jun 24 '25 13:06 bugfolder

...perhaps it would be better to use core icons anyhow.

Like arrow-circle-down-fill.svg 😀

indigoxela avatar Jun 24 '25 15:06 indigoxela

I'm not seeing this problem anymore?

Image

jenlampton avatar Sep 03 '25 19:09 jenlampton

I saw this problem this morning.

Image

irinaz avatar Sep 03 '25 19:09 irinaz

Well the source code still says it's font awesome, so I'll switch it over to Icon API anyway -- the new theme is doing that everywhere else. :)

Image

jenlampton avatar Sep 03 '25 19:09 jenlampton

I'm still seeing the empty square.

argiepiano avatar Sep 03 '25 19:09 argiepiano

Flushed all caches, including browser cache, still seeing this:

Image

Image

bugfolder avatar Sep 03 '25 20:09 bugfolder

I have just deployed the update. I'm seeing the new icon now. Image

I don't know why I could see the FontAwesome icon and no one else could -- @quicksketch thought maybe I have FontAwesome as a system font on my computer or something? I'm going to investigate that now...

jenlampton avatar Sep 03 '25 20:09 jenlampton

LGTM!

bugfolder avatar Sep 03 '25 20:09 bugfolder

All projects seem to be using the new icon except the Backdrop core project itself: https://backdropcms.org/project/backdrop

I'm getting a solid purple square, it seems like the icon is not defined in the CSS.

quicksketch avatar Sep 04 '25 17:09 quicksketch