diff options
author | Jaap Keuter <jaap.keuter@xs4all.nl> | 2016-10-04 19:55:12 +0200 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2016-10-07 22:50:34 +0000 |
commit | dd589f5c169f0d9feb55d86d4bc3fe3b991ea822 (patch) | |
tree | a6e73110a9728611be949b4d33a278c91b9aaa01 /ui/qt | |
parent | 41fe95fd3e865638db955d95a2e53fd746d5f775 (diff) | |
download | wireshark-dd589f5c169f0d9feb55d86d4bc3fe3b991ea822.tar.gz |
Qt: valign text in RTP Stream Analysis window.
The text on the left hand side of the RTP Stream Analtsis window is
a (HTML) table of table headers followed by table data. For some reason
the table data is center aligned in the cells of the table. This does not
line up with the text in the table header cells.
See http://imgur.com/a/QIQFd
Also the HTML document is not compliant. Closing tags are missing, for
td and html itself.
Change-Id: I1cd58e8702db12433c863757141a6b68c1e2550c
Reviewed-on: https://code.wireshark.org/review/18074
Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/qt')
-rw-r--r-- | ui/qt/rtp_analysis_dialog.cpp | 48 |
1 files changed, 24 insertions, 24 deletions
diff --git a/ui/qt/rtp_analysis_dialog.cpp b/ui/qt/rtp_analysis_dialog.cpp index eb72dc5d2e..7b67631bb2 100644 --- a/ui/qt/rtp_analysis_dialog.cpp +++ b/ui/qt/rtp_analysis_dialog.cpp @@ -900,7 +900,7 @@ void RtpAnalysisDialog::updateStatistics() r_clock_drift = (r_total_nr * r_sumtTS - r_sumt * r_sumTS) / (r_total_nr * r_sumt2 - r_sumt * r_sumt); } - QString stats_tables = "<html><head></head><body>\n"; + QString stats_tables = "<html><head><style>td{vertical-align:bottom;}</style></head><body>\n"; stats_tables += QString("<p>%1:%2 " UTF8_LEFT_RIGHT_ARROW) .arg(address_to_qstring(&src_fwd_, true)) .arg(port_src_fwd_); @@ -909,61 +909,61 @@ void RtpAnalysisDialog::updateStatistics() .arg(port_dst_fwd_); stats_tables += "<h4>Forward</h4>\n"; stats_tables += "<p><table>\n"; - stats_tables += QString("<tr><th align=\"left\">SSRC</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">SSRC</th><td>%1</td></tr>") .arg(int_to_qstring(ssrc_fwd_, 8, 16)); stats_tables += QString("<tr><th align=\"left\">Max Delta</th><td>%1 ms @ %2</td></tr>") .arg(fwd_statinfo_.max_delta, 0, 'f', 2) .arg(fwd_statinfo_.max_nr); - stats_tables += QString("<tr><th align=\"left\">Max Jitter</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Max Jitter</th><td>%1 ms</td></tr>") .arg(fwd_statinfo_.max_jitter, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Mean Jitter</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Mean Jitter</th><td>%1 ms</td></tr>") .arg(fwd_statinfo_.mean_jitter, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Max Skew</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Max Skew</th><td>%1 ms</td></tr>") .arg(fwd_statinfo_.max_skew, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">RTP Packets</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">RTP Packets</th><td>%1</td></tr>") .arg(f_total_nr); - stats_tables += QString("<tr><th align=\"left\">Expected</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">Expected</th><td>%1</td></tr>") .arg(f_expected); - stats_tables += QString("<tr><th align=\"left\">Lost</th><td>%1 (%2 %)</tr>") + stats_tables += QString("<tr><th align=\"left\">Lost</th><td>%1 (%2 %)</td></tr>") .arg(f_lost).arg(f_perc, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Seq Errs</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">Seq Errs</th><td>%1</td></tr>") .arg(fwd_statinfo_.sequence); - stats_tables += QString("<tr><th align=\"left\">Duration</th><td>%1 s</tr>") + stats_tables += QString("<tr><th align=\"left\">Duration</th><td>%1 s</td></tr>") .arg(f_duration / 1000.0, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Clock Drift</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Clock Drift</th><td>%1 ms</td></tr>") .arg(f_duration * (f_clock_drift - 1.0), 0, 'f', 0); - stats_tables += QString("<tr><th align=\"left\">Freq Drift</th><td>%1 Hz (%2 %)</tr>") // XXX Terminology? + stats_tables += QString("<tr><th align=\"left\">Freq Drift</th><td>%1 Hz (%2 %)</td></tr>") // XXX Terminology? .arg(f_clock_drift * f_clock_rate, 0, 'f', 0).arg(100.0 * (f_clock_drift - 1.0), 0, 'f', 2); stats_tables += "</table></p>\n"; stats_tables += "<h4>Reverse</h4>\n"; stats_tables += "<p><table>\n"; - stats_tables += QString("<tr><th align=\"left\">SSRC</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">SSRC</th><td>%1</td></tr>") .arg(int_to_qstring(ssrc_fwd_, 8, 16)); stats_tables += QString("<tr><th align=\"left\">Max Delta</th><td>%1 ms @ %2</td></tr>") .arg(rev_statinfo_.max_delta, 0, 'f', 2) .arg(rev_statinfo_.max_nr); - stats_tables += QString("<tr><th align=\"left\">Max Jitter</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Max Jitter</th><td>%1 ms</td></tr>") .arg(rev_statinfo_.max_jitter, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Mean Jitter</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Mean Jitter</th><td>%1 ms</td></tr>") .arg(rev_statinfo_.mean_jitter, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Max Skew</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Max Skew</th><td>%1 ms</td></tr>") .arg(rev_statinfo_.max_skew, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">RTP Packets</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">RTP Packets</th><td>%1</td></tr>") .arg(r_total_nr); - stats_tables += QString("<tr><th align=\"left\">Expected</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">Expected</th><td>%1</td></tr>") .arg(r_expected); - stats_tables += QString("<tr><th align=\"left\">Lost</th><td>%1 (%2 %)</tr>") + stats_tables += QString("<tr><th align=\"left\">Lost</th><td>%1 (%2 %)</td></tr>") .arg(r_lost).arg(r_perc, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Seq Errs</th><td>%1</tr>") + stats_tables += QString("<tr><th align=\"left\">Seq Errs</th><td>%1</td></tr>") .arg(rev_statinfo_.sequence); - stats_tables += QString("<tr><th align=\"left\">Duration</th><td>%1 s</tr>") + stats_tables += QString("<tr><th align=\"left\">Duration</th><td>%1 s</td></tr>") .arg(r_duration / 1000.0, 0, 'f', 2); - stats_tables += QString("<tr><th align=\"left\">Clock Drift</th><td>%1 ms</tr>") + stats_tables += QString("<tr><th align=\"left\">Clock Drift</th><td>%1 ms</td></tr>") .arg(r_duration * (r_clock_drift - 1.0), 0, 'f', 0); - stats_tables += QString("<tr><th align=\"left\">Freq Drift</th><td>%1 Hz (%2 %)</tr>") // XXX Terminology? + stats_tables += QString("<tr><th align=\"left\">Freq Drift</th><td>%1 Hz (%2 %)</td></tr>") // XXX Terminology? .arg(r_clock_drift * r_clock_rate, 0, 'f', 0).arg(100.0 * (r_clock_drift - 1.0), 0, 'f', 2); - stats_tables += "</table></p></body>\n"; + stats_tables += "</table></p></body></html>\n"; ui->statisticsLabel->setText(stats_tables); |