Bug 158143 - RFE: when building the xconf file..use only similar aspect ratio display modes
RFE: when building the xconf file..use only similar aspect ratio display modes
Product: Fedora
Classification: Fedora
Component: rhpxl (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Chris Lumens
: EasyFix, Patch
Depends On:
Blocks: FC5Target
  Show dependency treegraph
Reported: 2005-05-18 22:14 EDT by Jef Spaleta
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2006-02-01 10:08:35 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
patch to merge only same aspect ratio modes into xconf file (1.08 KB, patch)
2005-05-18 22:15 EDT, Jef Spaleta
no flags Details | Diff

  None (edit)
Description Jef Spaleta 2005-05-18 22:14:01 EDT
Description of problem:
All available modes smaller than selected resolution are included in the
xorg.conf modes line in the display section when using tools such as

It might be better to only include modes of the same aspect ratio, to prevent
unexpected situations like wide-screen modes being used if the selected high
resolution 4/3 aspect ratio mode is thrown out by X

I'm attaching a patch of the /usr/lib/python2.4/site-packages/rhpl/xhwstate.py
file which introduces a new compare_aspectratio  function and makes a small
change to the merge_into function to use compare_aspectratio when building the
mode list.

These changes do not affect the available resolution list as seen by
system-config-packages. You can still select a widescreen mode on a standard
moniter.  This patch, afaik, only affects what is written to the xorg.conf file
so that all modes written to the file have the same aspectratio as the
resolution 'selected'

This falls completely inside the realm of feature request.. so i don't expect
anyone to look at this till after fc4 ships. But since i was already looking at
the xhwstate.py to fix the resolution sort bug i thought I'd throw out this
quick feature patch and get it filed while I'm thinking about it. 

So take it for a spin when you have time to look at it and see if it makes sense
from a user expectation/usability perspective.

Comment 1 Jef Spaleta 2005-05-18 22:15:44 EDT
Created attachment 114546 [details]
patch to merge only same aspect ratio modes into xconf file

i rememebered to use diff -u this time... i deserve a beer
Comment 2 Jeremy Katz 2005-05-23 10:59:42 EDT
Looks sane, but going to hold off for now since we should be locking down pretty
hard for FC4 at this point.
Comment 3 Jef Spaleta 2005-05-23 11:07:51 EDT
yes, hold off. 

One thing I'm concerned about is whether or not widescreen displays should keep
a fallback low resolution standard mode like  "640x480" in the mode list or not.
 Send me a wide screen monitor to test and I'll examine the issue of fallback
modes in more detail.


Note You need to log in before you can comment on or make changes to this bug.