Bug #2763
closed
Dynamic Firewall logs doesn't add new rows
Added by John Bayly almost 12 years ago.
Updated almost 12 years ago.
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
/usr/local/www/diag_logs_filter_dynamic.php
- Modified to render the log in tables rather than divs
/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
/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)
/usr/local/www/css/table.css
- Added ellipsis css class that provides truncating cell contents.
Files
Widget is working fine, but there is something wrong with fonts and sizes as you can see on attached screenshot.
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.
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.
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.
- % Done changed from 0 to 100
- Status changed from Feedback to Closed
Also available in: Atom
PDF