Uploaded image for project: 'Observium'
  1. Observium
  2. OBS-515

Device App page highlights all navbar links of same app type

Details

    • Bug
    • Resolution: Fixed
    • Trivial
    • None
    • None
    • None
    • None

    Description

      If there are multiple instances of an application type for a device, the Apps page per device will set the navbar link to active on all links of that application type.

      Current Behavior:

      Expected Behavior:

      apps.inc.php.patch resolves issue.

      Attachments

        Activity

          [OBS-515] Device App page highlights all navbar links of same app type
          landy Mike Stupalov made changes -
          Workflow Original: classic default workflow [ 11214 ] New: Observium workflow [ 13838 ]
          adama Adam Armstrong made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          adama Adam Armstrong made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          adama Adam Armstrong made changes -
          Attachment Original: apps.inc.php-003.patch [ 11360 ]
          adama Adam Armstrong added a comment -

          how come there's an instance with no instance name?

          adama Adam Armstrong added a comment - how come there's an instance with no instance name?
          highonmikey Mikey made changes -
          Attachment Original: apps.inc.php.patch [ 11312 ]
          highonmikey Mikey made changes -
          Attachment Original: apps_inc_php-002.patch [ 11358 ]
          highonmikey Mikey made changes -
          Attachment New: apps.inc.php-003.patch [ 11360 ]
          highonmikey Mikey added a comment -

          Reviewed it since I'm not 330am tired. Actually seems fine, 003 is slightly better code, though. Added comments in there so you know wtf I'm doing.

          http://i.imgur.com/hiF98rO.png

          highonmikey Mikey added a comment - Reviewed it since I'm not 330am tired. Actually seems fine, 003 is slightly better code, though. Added comments in there so you know wtf I'm doing. http://i.imgur.com/hiF98rO.png
          highonmikey Mikey made changes -
          Attachment New: apps_inc_php-002.patch [ 11358 ]

          People

            adama Adam Armstrong
            highonmikey Mikey
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: