OBM
  1. OBM
  2. OBMFULL-5900

Huge printer with new menubar deactivated

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Normal Normal
    • Resolution: Fixed
    • Affects Version/s: 3.0.0
    • Fix Version/s: 3.0.0
    • Component/s: ui
    • Labels:
      None
    • Environment:

      OBM 3.0.0~0.alpha0+git20140408-1536-a981303 in Chromium

    • Similar Issues:
      Show 10 results 
    • Rank:
      6114

      Description

      Uncheck "Enable the unified OBM Topbar" and open the calendar, there is a huge printer icon (see screenshot)

        Activity

        Hide
        David Dolcimascolo added a comment -

        To QA:

        • Impacts obm-ui, I simply resized the image.
        • To test, well... Open your eyes

        Tested with FF 28 and Chrome 34 on Wheezy.

        Show
        David Dolcimascolo added a comment - To QA: Impacts obm-ui , I simply resized the image. To test, well... Open your eyes Tested with FF 28 and Chrome 34 on Wheezy.
        Hide
        Jenkins Continuous Integration Server added a comment -

        SUCCESS: Integrated in poll-next-30 472-6f4c41712e7aaff80b28b1ac4e79d5402b2f8a43
        OBMFULL-5900 Resized new printer image to 25x25. (jkornobis: 6f4c41712e7aaff80b28b1ac4e79d5402b2f8a43)

        • ui/resources/themes/default/images/ico_print.png
        • ui/resources/themes/default/styles/topbar.css
        Show
        Jenkins Continuous Integration Server added a comment - SUCCESS: Integrated in poll-next-30 472-6f4c41712e7aaff80b28b1ac4e79d5402b2f8a43 OBMFULL-5900 Resized new printer image to 25x25. (jkornobis: 6f4c41712e7aaff80b28b1ac4e79d5402b2f8a43 ) ui/resources/themes/default/images/ico_print.png ui/resources/themes/default/styles/topbar.css
        Hide
        Stephane COLSON added a comment -

        OK with version 3.0.0-0.alpha0+git20140423_1640_67a1273
        Tested with CentOS, Chromium and Iceweasel and IE11

        Show
        Stephane COLSON added a comment - OK with version 3.0.0-0.alpha0+git20140423_1640_67a1273 Tested with CentOS, Chromium and Iceweasel and IE11
        Hide
        Stephane COLSON added a comment -

        Arg nok, I now have 2 printers icons on the same page (see 2printers.png)

        Show
        Stephane COLSON added a comment - Arg nok, I now have 2 printers icons on the same page (see 2printers.png)
        Hide
        David Dolcimascolo added a comment -

        Ahem. The too printers were there before my patch and this Jira never talked about removing the second one... Is this a requirement?

        Show
        David Dolcimascolo added a comment - Ahem. The too printers were there before my patch and this Jira never talked about removing the second one... Is this a requirement?
        Hide
        Stephane COLSON added a comment -

        We don't need anymore the printer icon close to disconnect. With version 2.5.7, there was only one printer, so yes this is a must-do for UX coherency.

        Show
        Stephane COLSON added a comment - We don't need anymore the printer icon close to disconnect. With version 2.5.7, there was only one printer, so yes this is a must-do for UX coherency.
        Hide
        Jenkins Continuous Integration Server added a comment -

        SUCCESS: Integrated in poll-next-30 481-0ea9a5459bed4a72729a2f0c352d65a97954108d
        OBMFULL-5900 Printer icon well displayed both new and old top bar (jkornobis: 0c225bee02aab0cb39208bcd7366f3b6d46e36e1)

        • ui/resources/themes/default/styles/style.css
        • ui/obminclude/of/of_display.inc
        • ui/resources/themes/default/images/ico_print.png
        • ui/resources/themes/default/styles/topbar.css
        Show
        Jenkins Continuous Integration Server added a comment - SUCCESS: Integrated in poll-next-30 481-0ea9a5459bed4a72729a2f0c352d65a97954108d OBMFULL-5900 Printer icon well displayed both new and old top bar (jkornobis: 0c225bee02aab0cb39208bcd7366f3b6d46e36e1 ) ui/resources/themes/default/styles/style.css ui/obminclude/of/of_display.inc ui/resources/themes/default/images/ico_print.png ui/resources/themes/default/styles/topbar.css
        Hide
        Stephane COLSON added a comment -

        OK with version 3.0.0-0.alpha0+git20140512_1128_0ea9a54 on CentOS 6
        Tested with IE11, FF and chromium

        Show
        Stephane COLSON added a comment - OK with version 3.0.0-0.alpha0+git20140512_1128_0ea9a54 on CentOS 6 Tested with IE11, FF and chromium

          People

          • Assignee:
            jeremie kornobis
            Reporter:
            Stephane COLSON
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: