Project

General

Profile

Bug #5036

missing easy rule and DNS icons on diag_logs_filter.php

Added by Chris Buechler almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Chris Buechler
Target version:
-
Start date:
08/26/2015
Due date:
% Done:

100%

Spent time:
0.50 h
Affected version:
Affected Architecture:

Description

where the icons used to be there are just small blue rectangles

pfSense_localdomain_-_Status__System_logs__Firewall.jpg (80.4 KB) Steve Beaver, 08/28/2015 12:39 PM

Selection_015.png (7.88 KB) Chris Buechler, 09/01/2015 12:28 AM

Selection_016.png (4.43 KB) Chris Buechler, 09/01/2015 12:28 AM

Associated revisions

Revision 89a19fa1
Added by Steve Beaver almost 2 years ago

Fixed #5036

History

#1 Updated by Steve Beaver almost 2 years ago

Think this may be browser related and your browser doesn't understand that glyph-icon.

Please confirm by trying Firefox and comparing it with attached image.

I can change to a more browser friendly icon set.

#2 Updated by Chris Buechler almost 2 years ago

Yeah that's it. That's not very compatible cross-platform. Chrome on Linux and Windows (at a minimum) shows like the attached 016. Firefox on latest Ubuntu Linux shows like 015. Opera is similar to Chrome. Only thing I see where it works as designed is Firefox and IE on Windows and Firefox on OS X. We have double the percentage of Linux users (10+%) as IE users (~5%) on our sites, and Chrome is the #1 browser of our website visitors by a long shot (55%, Firefox second at 25%). Linux is a couple percentage points more popular than OS X. As is, this is broken for about 80% of the combinations of OS and browser that visit our forum.

Need something that works with all the major browsers, on Windows, Linux and OS X.

#3 Updated by Steve Beaver almost 2 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Steve Beaver to Chris Buechler

Icons replaced with Bootstrap glyphicons and key added at bottom of page.
Fixed Javascript so "resolve" function works correctly.

#4 Updated by Steve Beaver almost 2 years ago

  • % Done changed from 0 to 100

#5 Updated by Chris Buechler almost 2 years ago

  • Status changed from Feedback to Resolved

that works on Windows, Mac and Linux now, looks good.

Also available in: Atom PDF