Closed Bug 278533 Opened 20 years ago Closed 19 years ago

Preferences window too small for Composer prefs


(SeaMonkey :: Preferences, defect)

Not set


(Not tracked)



(Reporter: stefanh, Assigned: iannbugzilla)



(Keywords: regression)


(5 files, 1 obsolete file)

Mozilla 2005011406, Mac OS 10.3.7

The fix for bug 92686 added new prefs in pref-composer.xul. Unfortunately, the
pref windows height isn't enough, the checkbox "Return in a paragraph..." is cut
off at the bottom.
It happens on Windows too. Can someone change Hardware -> All and OS -> All?
This is theme dependent, on some themes it just about fits on (Classic on Windows)
OS: MacOS X → All
Hardware: Macintosh → All
Why is it important to have a user visible preferences option for the maximum
number of pages listed in the "Recent Pages" menu? Could we perhaps just remove
this option from the pref pageto fix this bug? A hidden pref with a reasonable
default would be enough for this IMHO.
One reason to keep the pref for "open recent" exposed is because if someone
wants to open a file and not have it listed in the prefs it is easy to tell them
to set the value to 0 and then reset after they have edited the file.

I don't really have a good sense of how many people actually change the value. 
I know I do occasionally (when I have a huge number (~20) of files I'm editing
and want to keep in the list but generally I don't like the list too big (8-10).

If we are going to remove a pref (or two), we should also consider the table
(maintain table structure) pref as well as the two radio buttons (which don't
work?) for reformatting the source when the file is saved.

If we don't want to remove a pref, we can combine the radio buttons into a
dropdown list or move one or more prefs to other panels.
Flags: blocking1.8b?
not going to block on this for beta but we'd take a reviewed patch into beta2
and possibly beyond as long as it didn't add or modify localizable strings. 
Flags: blocking1.8b? → blocking1.8b-
*** Bug 285290 has been marked as a duplicate of this bug. ***
The "Table Editing" and "Cascading Style Sheet (CSS) Editing" group boxes
combined into one "Editing" group box
The "Table Editing", "Cascading Style Sheet (CSS) Editing" and "Behavior of the
return key" group boxes combined into one "Editing" group box
Since all three checkboxes deals with editing, I'll vote for proposal B ;)
Flags: blocking-seamonkey1.0a?
Proposal 2 sounds good to me...
While we're at it, we should probably even replace the remaining radio with a
"Retain original source formatting" checkbox, as it's pretty obvious that the
other possible variant is reformatting the source, and it's bad style to have a
two-way radio where one choice is obvious from knowing the other one.
A checkbox is cleaner and simpler.
Add my vote for proposal B.
Changes from Proposal B
* Changed radio group to single checkbox as suggested by Kairo
* Changed "Retain" to "Preserve" in new checkbox label
* Added accesskeys where they were missing
Assignee: prefs → bugzilla
Attached patch Patch for Proposal C v0.1c (obsolete) — Splinter Review
Implements Proposal C
Attachment #179376 - Flags: review?(brade)
Comment on attachment 179376 [details] [diff] [review]
Patch for Proposal C v0.1c

Looks OK except you forgot to reverse the formatting checkbox.
Attachment #179376 - Flags: review?(brade)
Changes since v0.1c
* Added reversed="true" to new checkbox - thanks Neil for the catch
Attachment #179376 - Attachment is obsolete: true
Attachment #179501 - Flags: review?(brade)
Attachment #179501 - Flags: review?(brade) → review?(daniel)
Comment on attachment 179501 [details] [diff] [review]
Revised patch for proposal C v0.1ca (Checked in)

Does the pref for maintaining original source formatting actually work now?  At
one point (years ago?) the checkbox and text weren't in synch with each other
and it's not clear if that is fixed with this patch or was fixed long ago or ?
brade, from my testing using the "Preserve orginal source formatting" does make
a difference to the HTML source code that you see in composer.
Comment on attachment 179501 [details] [diff] [review]
Revised patch for proposal C v0.1ca (Checked in)

marking daniel's r= and requesting sr=
Attachment #179501 - Flags: superreview?(
Attachment #179501 - Flags: review?(daniel)
Attachment #179501 - Flags: review+
Attachment #179501 - Flags: superreview?( → superreview+
Comment on attachment 179501 [details] [diff] [review]
Revised patch for proposal C v0.1ca (Checked in)

Checking in composer/content/pref-composer.xul;
/cvsroot/mozilla/editor/ui/composer/content/pref-composer.xul,v  <-- 
new revision: 1.34; previous revision: 1.33
Checking in locales/en-US/chrome/composer/pref-composer.dtd;
<--  pref-composer.dtd
new revision: 1.2; previous revision: 1.1
Attachment #179501 - Attachment description: Revised patch for proposal C v0.1ca → Revised patch for proposal C v0.1ca (Checked in)
Closed: 19 years ago
Resolution: --- → FIXED
v fixed (2005040610, Mac OS X)
v fixed (2005040610, Mac OS X)
Flags: blocking-seamonkey1.0a?
You need to log in before you can comment on or make changes to this bug.