Rainie3535 / sigil

Automatically exported from code.google.com/p/sigil
GNU General Public License v3.0
0 stars 0 forks source link

Find and Replace minor interface changes #1218

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The new F&R is an improvement on the old one, although I think some minor 
tweaks might help make it a little easier for new and experienced users alike.  

To make it easier to include I've attached a patch with a description of all 
the changes below.  You may or may not want all of them exactly as they are, 
but I hope you'll find they make things just a little clearer and convenient 
for one of the main uses of Sigil.

* Changed the cursor positioning/scrolling so that the found text has space 
above/below if possible
* Changed look in All HTML Files to All HTML Files in Code View, and 
automatically switch to code view if in Book View.  Prevents the annoying popup 
without needing a preference and clearly tells the user what will happen.  
Could try to stay in split view instead of switching to full code view.  Could 
change routine from being boolean but left in case there is a need for other 
logic
* Changed tab order so tab from find moves to the replace box - basically makes 
it sort of top to bottom/left to right so you can still tab to any entry in a 
sensible order
* Added shortcuts for Replace All (Alt-A) and Count All (Alt-C)
* Added menu items Replace Next/Previous, Replace All, Count All
* Changed the Find Next, Replace Next arrows to text buttons to make them 
easier to see and understand.  Left the previous buttons as arrows since making 
them text  made the box look  too busy and they aren't used as frequently - but 
maybe part way isn't good enough and it should be all text
* Changed All and Count to Replace All and Count All for clarity
* Added longer tooltips to some buttons
* Changed the show message text to be singular/plural/none as appropriate, and 
removed "." at end as unnecessary
* Changed Look to Look in with some extra spacing for readability

Original issue reported on code.google.com by meme90...@gmail.com on 30 Jan 2012 at 6:17

Attachments:

GoogleCodeExporter commented 9 years ago
Merged.

The following changes were not included:

> * Changed Look to Look in with some extra spacing for readability

The spacing was set to be a bit squashed on purpose. I have an 11" screen and 
keeping it as close as possible lets me make the window smaller than full 
screen.

> * Changed the Find Next, Replace Next arrows to text buttons to make them 
easier to see and understand.  Left the previous buttons as arrows since making 
them text  made the box look  too busy and they aren't used as frequently - but 
maybe part way isn't good enough and it should be all text

Again, 11" screen. Being able to make the window smaller is a must. Also, only 
the next buttons having text looked odd next to the previous buttons as arrows.

> * Added longer tooltips to some buttons

They were confusing a bit confusing.

> * Changed the show message text to be singular/plural/none as appropriate

They are plural strings for translation. I've just been lazy and haven't wanted 
to make an English translation for two strings. Once a tr file is created they 
will look right and use the appropriate text for the given number.

Original comment by john@nachtimwald.com on 31 Jan 2012 at 1:11

GoogleCodeExporter commented 9 years ago

Original comment by john@nachtimwald.com on 31 Jan 2012 at 2:03