Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-21928

Redesign of results page

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: cppcheck-plugin
    • Labels:
      None
    • Environment:
      All, plugin version 1.14
    • Similar Issues:

      Description

      There are several issues on the results page:

      • All tables have 100% width of the page.
      • It would be better if summary table has rows instead of columns.
      • Errors trend is not necessary, it can be part of summary table (footer).
      • Column headers in details table look more like identifiers than words.
      • Text in details table is not wrapping, so the page contains unnecessary horizontal scroll bar.

        Attachments

          Activity

          Show
          mixalturek Michal Turek added a comment - Implemented in https://github.com/mixalturek/cppcheck-plugin/commit/9335f6c706ef9afc2887411320bc306fe4d32a32 .
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Michal Turek
          Path:
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckResult.java
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/statistics.jelly
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/summary.jelly
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/summary.jelly
          http://jenkins-ci.org/commit/cppcheck-plugin/9335f6c706ef9afc2887411320bc306fe4d32a32
          Log:
          JENKINS-21928 Redesign of results page

          • Table in buildaction/summary.jelly extracted to a separate file and reused in the results page. This table uses rows instead of columns so it is better readable and supports sorting, it also contains more information (delta).
          • "Errors Trend" removed, it is part of the summary.
          • "Cppcheck Engine versions" updated from one cell table to a simple paragraph.
          • summary.jelly renamed to details.jelly, the file contains details and not summary.
          • Wrapping of long lines is now possible in the details table to remove horizontal scroll bar.
          • Column headers renamed in the details table to look more like words instead of identifiers.
          • The tables have their widths instead of 100% width of the page.
          • Missing support for translations ${%Text} implemented.
          • Function getNumberNewErrorsFromPreviousBuild() is now unused so it was removed.
          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckResult.java src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/statistics.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/summary.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/summary.jelly http://jenkins-ci.org/commit/cppcheck-plugin/9335f6c706ef9afc2887411320bc306fe4d32a32 Log: JENKINS-21928 Redesign of results page Table in buildaction/summary.jelly extracted to a separate file and reused in the results page. This table uses rows instead of columns so it is better readable and supports sorting, it also contains more information (delta). "Errors Trend" removed, it is part of the summary. "Cppcheck Engine versions" updated from one cell table to a simple paragraph. summary.jelly renamed to details.jelly, the file contains details and not summary. Wrapping of long lines is now possible in the details table to remove horizontal scroll bar. Column headers renamed in the details table to look more like words instead of identifiers. The tables have their widths instead of 100% width of the page. Missing support for translations ${%Text} implemented. Function getNumberNewErrorsFromPreviousBuild() is now unused so it was removed.
          Hide
          mixalturek Michal Turek added a comment -

          Implemented, will be released in version 1.15.

          Show
          mixalturek Michal Turek added a comment - Implemented, will be released in version 1.15.

            People

            • Assignee:
              mixalturek Michal Turek
              Reporter:
              mixalturek Michal Turek
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: