[dev.icinga.com #835] Avoid the horizontal scroll bar at the bottom of the menu #394

Closed
icinga-migration opened this Issue Sep 29, 2010 · 5 comments

Projects

None yet

1 participant

@icinga-migration
Member

This issue has been migrated from Redmine: https://dev.icinga.com/issues/835

Created by tontonitch on 2010-09-29 15:42:17 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2010-09-30 13:35:45 +00:00)
Target Version: 1.2 (Stable)
Last Update: 2014-12-08 09:34:19 +00:00 (in Redmine)


Hi,

For when I started to work with Nagios some years ago, an horizontal scroll bar has been a bit annoying. And it's also the case on Icinga (according to the last git version).

May I propose a little change in the index.html file:

  • enlarge a little bit the frame for the menu from 210 to 220
  • set scrolling="auto" to only keep the vertical scroll bar

Attached a patch for that tiny change.

Regards,

Yannick

Attachments

Changesets

2010-09-30 12:33:39 +00:00 by mfriedrich b1d8b96

classic ui: avoid the horizontal scroll bar at the bottom of the menu (Yannick Charton) #835

fixes #835
Member

Updated by mfriedrich on 2010-09-29 16:37:34 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mfriedrich

thanks. i'll have a look later on.

Member

Updated by mfriedrich on 2010-09-30 12:30:19 +00:00

215 fits better than 220, if having a scrollbar. rest will be in git shortly, thx.

Member

Updated by tontonitch on 2010-09-30 13:26:07 +00:00

For me, 215 is not enough. The horizontal scrollbar disappears from 217. I proposed 220 to fit to almost all environments. Tested on firefox and IE.

Member

Updated by mfriedrich on 2010-09-30 13:35:45 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset commit:"b1d8b96e6475e344ab9f0177704bf1ff24011145".

Member

Updated by mfriedrich on 2014-12-08 09:34:19 +00:00

  • Project changed from 19 to Core, Classic UI, IDOUtils
  • Category changed from 44 to Classic UI
@icinga-migration icinga-migration added this to the 1.2 (Stable) milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment