Project

General

Profile

Actions

Bug #2763

closed

Dynamic Firewall logs doesn't add new rows

Added by John Bayly almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Normal
Category:
Web Interface
Target version:
Start date:
01/18/2013
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
Affected Architecture:

Description

The Dynamic Firewall Log page doesn't add rows, it only updates the php generated rows which means that if it's loaded after the log is cleared, it'll never show any activity. The same occurs for the log widget.

It also uses divs and absolutely sized spans instead of tables like the other log pages, meaning that the dynamic log can't change with variable width pages.

I've patched 4 files to make
  1. /usr/local/www/diag_logs_filter_dynamic.php - Modified to render the log in tables rather than divs
  2. /usr/local/www/javascript/filter_log.js - Modified to support addition or insertion of rows, so that the table will grow to the number of rows specified by nentries
  3. /usr/local/www/widgets/widgets/log.widget.php - Modified to render the log in tables rather than divs, this uses a fixed table layout and truncates the log info with an ellipsis (title attributes are used to display the full cell text)
  4. /usr/local/www/css/table.css - Added ellipsis css class that provides truncating cell contents.

Files

dynamic_log.diff (13.4 KB) dynamic_log.diff John Bayly, 01/18/2013 06:03 AM
log_widget.png (11.5 KB) log_widget.png Renato Botelho, 01/24/2013 07:16 PM
Widget - Modified.png (12 KB) Widget - Modified.png John Bayly, 01/25/2013 04:45 AM
log_widget_with_align.png (13.2 KB) log_widget_with_align.png Renato Botelho, 01/25/2013 06:01 AM
log_widget_without_align.png (12.9 KB) log_widget_without_align.png Renato Botelho, 01/25/2013 06:01 AM
Actions #1

Updated by Renato Botelho almost 9 years ago

Widget is working fine, but there is something wrong with fonts and sizes as you can see on attached screenshot.

Actions #2

Updated by John Bayly almost 9 years ago

I'm guessing that you mean the column headers? I just copied across the span width values onto the column headers, but those widths don't work so cell with table cells. On a second look, the percentages didn't even add up to 100%!

How does it look if the column widths are modified as such:

    <colgroup>
        <col style="width: 8%;">
        <col style="width: 10%;">
        <col style="width: 35%;">
        <col style="width: 35%;">
        <col style="width: 12%;">
    </colgroup>

If you mean the cell values being truncated, the problem I was having with the original Widget was that the contents overflowed into each other - making them unreadable. Hovering the mouse over the entry should display the full cell value.

Actions #3

Updated by Renato Botelho almost 9 years ago

Column widths looks good with new values, but rows height doesn't. Take a look at new attachments, seems the Act icon is not vertically aligned. Removing align="middle" from <a> improves it a bit.

Actions #4

Updated by John Bayly almost 9 years ago

I'd agree with that, the reason I hadn't noticed was that in [my version] of Chrome, I don't see the alignment problem, but I do in Firefox. I took that html from diag_logs_filter.php, and I see the same rendering differences between Firefox and Chrome on diag_logs_filter.php too. How about yourself?

My suggestion would be to remove the align="middle" attribute from all the filter log pages.

Actions #5

Updated by Renato Botelho almost 9 years ago

  • % Done changed from 0 to 100
Actions #6

Updated by Renato Botelho almost 9 years ago

  • Status changed from Feedback to Closed
Actions

Also available in: Atom PDF