Having uploaded fonts to the local, it is not clear how to delete them. After going to the local fonts directory, physically removing the folder, and clearing cache, the font continues to show under the browse tab. What is the recommended way to remove local fonts?

CommentFileSizeAuthor
#11 views-interface.jpg42.85 KBBarisW
#2 AdministrationImportFonts.jpg74.25 KBkevin-bcr
#2 AppearanceApplyFonts.jpg126.74 KBkevin-bcr
#2 AppearanceBrowseFonts.jpg159.57 KBkevin-bcr

Comments

sreynen’s picture

sreynen commented

There should be a delete button on the font edit screen, which is a bit hidden in the new UI. When you hover over a font in the browse interface, you should see a little gear icon you can click to go to the edit page.

We should probably make this easier to get to, but I'm not sure where it would be useful to add a link. Where did you look for it when you were looking? That's probably where we should put it.

kevin-bcr’s picture

kevin-bcr commented
StatusFileSize
new AppearanceBrowseFonts.jpg159.57 KB
new AppearanceApplyFonts.jpg126.74 KB
new AdministrationImportFonts.jpg74.25 KB

sreynen: Thank you for the reply and help. You know, I'm hovering all over the place and I just cannot get a gear to show up (except for the one to "Edit View" right next to the browse button). If I'm correct, there are three possible screens to look for (I've included a photo of each so you can see what I'm seeing).
1. Administration > Import Fonts
2. Appearance > Apply Fonts
3. Appearance > Browse Fonts

It seems most intuitive to me to go to either Administration where I imported the fonts and to see, perhaps, another tab with list of all imported fonts and a button to delete them OR to go to the Appearance section and always have another link next to enable/disable that read remove -- whether on the apply fonts or the browse fonts page.

Any way, again, I see no gear. So, maybe I'm there's a fourth page I should be looking at? On my dev server I have 7.x-2.0-dev and on my production server I have 7.x-2.0-rc1. They look the same.

Anyway, I've attached pictures. I'm trying to delete the one that is currently shown as disabled. As I said earlier, I have manually deleted the folder already. Hope that doesn't cause a problem.

Thanks for your help and all your work.
Kevin

sreynen’s picture

sreynen commented
Version: 7.x-2.0-rc1 » 7.x-2.x-dev
Assigned: Unassigned » sreynen
Category: support » bug

The hover gear should be showing up in that 3rd screen. Which browser are you using?

I'll work on adding a list of local fonts with edit and delete links to the configuration section.

Deleting the directory shouldn't cause any problems beyond that font no longer being usable.

kevin-bcr’s picture

kevin-bcr commented

sreynen: Checked again in Firefox 7.01, Chrome 15.0.874.106 m, and Internet Explorer 9.0.8112.1621 64bit. I can find no gear, other than the one to edit the view, in Appearances > Browse screen. I'm moving the mouse all over and around the rectangle with the rounded corners. Thanks for your work!

BarisW’s picture

BarisW commented
Status: Active » Fixed

@sreynen: I don't see the gear as well. Might be that your view is overridden? Try reverting it to make sure we look at the same view.
@kevin-bcr: I've added a delete link to the Apply interface for local fonts (7.x-2.x-dev). Let me know if this works for you.

sreynen’s picture

sreynen commented
Title: Not clear how to remove local fonts » Font edit links not showing
Component: Documentation » Code (general)
Status: Fixed » Active

Oh, I don't see the gears either. Sorry, I should have checked that before describing where I thought they were. I'm going to keep this open while I fix that.

I might also work on listing local fonts in the configuration section, since the apply page requires first enabling a font before deleting it, a little strange.

BarisW’s picture

BarisW commented

@scott: I've made it so that the Apply page shows enabled fonts AND all local fonts (independent of their status). So that external fonts have a 'disable' link and local fonts have a 'delete' link. Now that I think about it, it might also be possible that one wants to disable a local font without removing it.

So for local fonts, I could show a link 'disable' if it's active, and a link 'delete' once its disabled. Would that work?

kevin-bcr’s picture

kevin-bcr commented

Hello everyone. I just downloaded the Nov. 9th Dev version and installed. All looks and works well. "I see gears" and "I see 'Delete' text." Thanks for the work.

I do have another question: I had already manually deleted the specific font folders under local_fonts. So, just curious: Does the 'Delete' button normally remove these folders or just the record in the database? It might be worth clarifying this for users when the confirm delete dialog box comes up.

Am I supposed to have just one folder in local_fonts for each font that I see in the Apply and browse tabs? I installed OpenSans in three weights and in both italic and normal -- a total of six fonts. But I also have an empty folder labeled "OpenSans." Is this required?

Again: Thank you. I enjoy what you've done.

Kevin

kingfisher64’s picture

kingfisher64 commented

It may be worth removing that rc1 version from the drupal list as the gear cog is not showing up (as stated in previous comments). I've just tried all 3 versions on drupal.org and 1.6 and the latest dev (27th Nov) work fine.

From a design point it would be better if "enable" as an option wasn't present on the font preview page until the selectors were entered. I kept clicking enable thinking that on the next screen there was going to be a option to set those selectors, and the cog kept going round and round - I thought the mod was broken. There could do with being text "configure" or the cog image or a bit brighter as I missed it for while.

The above is not criticism just suggestions - I don't want you to think all your hard work isn't appreciated. I think your mod is fantastic sreynen, thank you - I will be using it in all my sites. :)

sreynen’s picture

sreynen commented

We should have a final 2.0 release very soon. Until then, the missing gear icon doesn't strike me as critical functionality, so I'm okay leaving the rc1 release up.

I very much appreciate suggestions on the UI, as I'd like to make that as clear and quick as possible. I like the idea of, at least optionally, setting selectors directly from the browse interface.

BarisW’s picture

BarisW commented
StatusFileSize
new views-interface.jpg42.85 KB

kingfisher64: could you verify that your View is in its default state? See attachment.
If it is shown as 'Database overridden code', you might try to revert the view to its default state.

kingfisher64’s picture

kingfisher64 commented

I've not touched the views. it read "In code"

jonathan_hunt’s picture

jonathan_hunt commented

Using 7.x-2.0-rc1, my @font-your-face Browse view is "in code" and I see no "edit" gear anywhere (Chrome 16). Or any way to delete a local font once registered. That's quite confusing, so I went hunting and found this issue.

I just updated to 7.x-2.x-dev 2011-Dec-17 and I now see the gear icon. I'd recommend the UI was just a simple table with action links (similar to content management view etc.)

jonathan_hunt’s picture

jonathan_hunt commented

But I'm still mystified by how I delete a font via the UI. I've imported a local font with TTF and now I want to add EOT to it but there doesn't seem to be any way to do this...

sreynen’s picture

sreynen commented

jonathan_hunt, there should be a delete button on the edit screen. We previously had a simpler table view. You're the first person who has suggested that would be better than what we have now, so we're unlikely to change it back. But if you're familiar with Views, you can easily make your own version.

sreynen’s picture

sreynen commented
Status: Active » Fixed

This is fixed in the dev version and we just need to make a release now. I created a new issue for setting selectors from the browse interface:

#1397410: Add ability to set selectors from browse interface

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.