View Single Post
Old 09-26-2012, 06:14 PM   #86
Serpentine
Evangelist
Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.Serpentine ought to be getting tired of karma fortunes by now.
 
Posts: 416
Karma: 1045911
Join Date: Sep 2011
Location: Cape Town, South Africa
Device: Kindle 3
Oi! We needs a new beta to play with

Minor notes :
  • Could the image preview allow the information text (which is really nice!) to be selectable (Who's too lazy to remember two numbers? This guy!)
  • When removing an item for an epub, the action is to 'delete', however since you use 'add' for the converse operations; Surely 'remove' would be a better choice?

Default search presets :
  • There is a duplicated &shy replacement out of the replacement group it seems?
  • Drag and drop movement in the search editor seems like something bad waiting to happen to someone... up/down buttons or something might be better, but I have no idea about doing that with groups.
  • Empty paragraph - <p>&nbsp;*</p> - This should be <p>(&nbsp*</p> - or more likely something like <p\b[^>]*>(&(nb|em|en|thin)sp;|\s)*</p>
  • Join Paragraphs - (?sU)([A-Za-z,])</p>\s+<p[^>]*> - Would be better as ([[:alpha:],])</p>\s*<p\b[^>]*> - the replace also needs a space before the \1.

Nothing too fancy, if there's going to be a community regex pack or something, perhaps I can get out some of the good stuff.

Last edited by Serpentine; 09-26-2012 at 06:49 PM.
Serpentine is offline