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

Glyphs on version 2.3.2 do not work #4798

Closed
sensei116 opened this issue Jun 22, 2022 · 7 comments
Closed

Glyphs on version 2.3.2 do not work #4798

sensei116 opened this issue Jun 22, 2022 · 7 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@sensei116
Copy link

Bug Description

Glyph table is half-empty.
It looks that the issue is in "private function glyphTab($cat='', $parm=array())".
This function works fine for 2.3.0.

How to Reproduce

Steps to reproduce the behavior:

  1. Go to "Navigation" settings and try to create new menu link.
  2. Click on icon, select "glyphs" tab.
  3. See error

Expected Behavior

It will be good, if this works like it was in 2.3.0 - the full tab of glyph icons.

Screenshots

image

Server Information

PHP Operating System

Linux DCS-KAZLAB-ROOT 4.19.0-5-amd64 #1 SMP Debian 4.19.37-5+deb10u2 (2019-08-08) x86_64

PHP Version

PHP Version 7.3.31-1~deb10u1

PHP Modules

apache2handler
calendar
Core
ctype
curl
date
dom
exif
fileinfo
filter
ftp
gd
gettext
hash
iconv
json
libxml
mbstring
mysqli
mysqlnd
openssl
pcre
PDO
pdo_mysql
Phar
posix
readline
Reflection
session
shmop
SimpleXML
sockets
sodium
SPL
standard
sysvmsg
sysvsem
sysvshm
tokenizer
wddx
xml
xmlreader
xmlwriter
xsl
Zend OPcache
zlib

Client Information

Your user agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Firefox/91.0
Other HTTP headers
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.5
Host: duckduckgo.com
TE: trailers
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Firefox/91.0
SEC-FETCH-DEST: document
SEC-FETCH-MODE: navigate
SEC-FETCH-SITE: none
SEC-FETCH-USER: ?1
SSL-JA3-HASH: be12eb3bab3313e8d159fd00f469cae2
UPGRADE-INSECURE-REQUESTS: 1

@sensei116 sensei116 added the type: bug A problem that should not be happening label Jun 22, 2022
@Jimmi08
Copy link
Contributor

Jimmi08 commented Jun 22, 2022

Confirmed.
the reason is wrong markup for old glyphicons
<i class="glyphicon glyphicon-glyphicon-collapse-down"></i>
it should be
<i class="glyphicon glyphicon-gcollapse-down"></i>

For me bigger issue is mixed FA 5 a Glyphs icons under label bootstrap...

@sensei116
Copy link
Author

I am not sure if I have done the right thing, but if I remove this part (please see screenshot attached), then it looks finally better.
But it is strange, that in 2.3.2 we have less glyphs amount than it was in 2.3.0.

image

@Jimmi08
Copy link
Contributor

Jimmi08 commented Jun 23, 2022

Your fix looks as solution for admin.

Your frontend theme should add icons set too... because what is good to have there glyphicons set if your frontend theme doesn't support them and they will not be displayed? If there is the same icon name for FA 4 and FA 5 - f.e. envelope - you will not notice difference.

I am sorry, I had lost track about how glyphs work in e107 already...

@CaMer0n
Copy link
Member

CaMer0n commented Jul 1, 2022

@sensei116 Thank you for posting this issue. What theme are you using? If it's not from the e107 release, could you please post the contents here of your theme.xml

@sensei116
Copy link
Author

sensei116 commented Jul 1, 2022 via email

@CaMer0n CaMer0n added this to the e107 2.3.3 milestone Jul 1, 2022
@CaMer0n
Copy link
Member

CaMer0n commented Jul 1, 2022

Thank you @sensei116, much appreciated!

When I install VOUX, I only get the FontAwesome 4 glyphs in the admin area (which is the intended behavior). I see from your theme.xml file that you should be seeing FontAwesome 5 glyphs, so I'm not sure why Bootstrap glyphs are appearing for you. Is it possible you have something like this in your theme.php file? :

define("FONTAWESOME", X);

If so, you can remove it.

@sensei116
Copy link
Author

sensei116 commented Jul 11, 2022

Hi Cameron.

Yep. I have so:
image

When I remove it I see glyphs v.5 anyway:
image

But it can be related to changes that I have made previously to fix the issue. So, I am not 100% sure if it is the final solution or not.
But for now anything works fine, so e107 is the best again! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

3 participants