Closed (fixed)
Project:
Text Resize
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
15 Jan 2008 at 16:00 UTC
Updated:
31 Mar 2008 at 05:45 UTC
Increments of +1 / -1 are too small to notice.
It's better to have +2 / -2 increments to have a noticeable effect.
This also means the the default min and max options should be made to +5 and -1.
Let's allow to define font resize increments other then +1/-1.
For the meanwhile, here is a patch to fontsizer.js which defines the default increment to be 2, not 1. (you will have to set the min and max options accordingly -max +2 is too small).
37c37
< if(($.FontSizer.level) + 1 <= $.FontSizer.options.max) {
---
> if(($.FontSizer.level) + 2 <= $.FontSizer.options.max) {
40c40
< var next = (parseInt($.FontSizer.level) + 1);
---
> var next = (parseInt($.FontSizer.level) + 2);
46c46
< if(($.FontSizer.level - 1) >= $.FontSizer.options.min) {
---
> if(($.FontSizer.level - 2) >= $.FontSizer.options.min) {
49c49
< var next = (parseInt($.FontSizer.level) - 1);
---
> var next = (parseInt($.FontSizer.level) - 2);
Amnon
-
Professional: Drupal Israel | Drupal Development & Consulting | Eco-Healing | Effective Hosting Strategies | בניית אתרים
Personal: Hitech Dolphin: Regain Simple Joy :)
Comments
Comment #1
redbox2000@drupal.org commentedOk. Good idea. I will take this one step further and introduce a "step" value so we can go up/down in any increment. I will have this fixed and uploaded within next few days.
Comment #2
redbox2000@drupal.org commentedOk. Done! Check out the new demo page for how this module can be used.
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.