Mailinglist Archive: opensuse-buildservice (258 mails)

< Previous Next >
[opensuse-buildservice] [PATCH] Graphs in monitor page overflow from the main content area
  • From: "Srinidhi B" <srinidhi@xxxxxxxxxx>
  • Date: Sun, 08 Aug 2010 10:48:46 -0600
  • Message-id: <4C5F2D46020000550005F790@xxxxxxxxxxxxxxxxxxxxx>
Hi,

In the OBS (2.0.5) monitor page, the graphs overflow from the main content area:

https://build.opensuse.org/monitor

Here's a small patch to define correct width and height so that they stay
inside the content area.

Srinidhi.

diff --git a/src/webui/app/views/monitor/_graphs_arch.rhtml
b/src/webui/app/views/monitor/_graphs_arch.rhtml
index f9f0eb5..9d98455 100644
--- a/src/webui/app/views/monitor/_graphs_arch.rhtml
+++ b/src/webui/app/views/monitor/_graphs_arch.rhtml
@@ -2,15 +2,15 @@
<h1><%= title %></h1>
<ul class="monitorgraphs">
<li>
-<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'waiting', :range => 24, :time => Time.now.to_i
) %>" alt='Waiting' width="400" height="300"/>
+<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'waiting', :range => 24, :time => Time.now.to_i
) %>" alt='Waiting' width="300" height="250"/>
<span>Waiting to build</span>
</li>
<li>
-<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'blocked', :range => 24, :time => Time.now.to_i
) %>" alt='Blocked' width="400" height="300"/>
+<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'blocked', :range => 24, :time => Time.now.to_i
) %>" alt='Blocked' width="300" height="250"/>
<span>Blocked by other packages</span>
</li>
<li>
-<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'workers', :range => 24, :time => Time.now.to_i
) %>" alt='Workers' width="400" height="300"/>
+<img class="monitorplot" src="<%= url_for(:controller => :monitor, :action =>
:plothistory, :set => prefix + 'workers', :range => 24, :time => Time.now.to_i
) %>" alt='Workers' width="300" height="250"/>
<span>Workers</span>
</li>
</ul>

--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups