Feature #188
Add debug to the scheme level
| Status: | Closed | Start: | 10/04/2009 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | 10/16/2009 | |
| Assigned to: | tshif | % Done: | 100% |
|
| Category: | - | |||
| Target version: | Caller ID Superfecta v 2.2.1 Maintenance Release | Estimated time: | 1.00 hour |
Description
The debug function will test only one scheme at this time (the one on the scheme page that is currently open).
I would suggest that we add a 'test all scheme' checkbox next to the debug button to be able test the number against the scheme matching in addition to only the source lookup process.
History
Updated by patrick_elx over 2 years ago
seems easy to do.
Just need to add a checkbox and in the function Ht_debug(thenumber)(in page.superfecta.php)
change the line
var poststr = "debug=yes&thenumber=" + thenumber+"&scheme=";
by
var poststr = "debug=yes&thenumber=" + thenumber;
and add to poststr "&scheme=" only if the checkbox is not checked.
Updated by tshif over 2 years ago
- Due date set to 10/16/2009
- Status changed from New to Assigned
- Assigned to set to patrick_elx
- Target version changed from Caller ID Superfecta - Future Versions to Caller ID Superfecta v 2.2.1 Maintenance Release
I agree. I set the due date to be the same as the projected release of 2.2.1 - let me know if that doesnt work for you.
Updated by patrick_elx over 2 years ago
I'm just not sure how to properly create a checkbox in php.. I'll have to look for it. I'll try to work on it tomorrow.
Updated by patrick_elx over 2 years ago
- Status changed from Assigned to QA Testing
- % Done changed from 0 to 80
- Estimated time set to 1.00
Updated by tshif over 2 years ago
- Assigned to changed from patrick_elx to tshif
Updated by tshif over 2 years ago
QS: FUnctions passed QS. Cosmetic Issues remain in fly by help.
dev: Can the flyby help be moved to the left? Currently, hoving over link causes browser to expand to the right. Moving flyby text to the left would correct. Can this be done?
Updated by patrick_elx over 2 years ago
I'm not sure...
However, it should be easy just to put the checkbox below the number box. That should center the flyby help.
Updated by tshif over 2 years ago
- Status changed from QA Testing to Assigned
- Assigned to changed from tshif to patrick_elx
Ok - great - you're the man!
Updated by tshif over 2 years ago
- Status changed from Assigned to QA Testing
- Assigned to changed from patrick_elx to tshif
Updated by patrick_elx over 2 years ago
please recheck rev152 there was a conflict with your changes rev 150 and mine rev151.
Updated by tshif over 2 years ago
I just double checked - when I pull r151 it works fine for me - what are you seeing as a failure?
I tested in IE8 and firefox.
Updated by patrick_elx over 2 years ago
The flyby was not working and the size of the police was not consistent with the rest of the page.
But again it could have been a conflict as I was working on 150 while you uploaded 151.
I can try to downgrade again to recheck, but 152 seems to work ok here.
Updated by tshif over 2 years ago
Updated by patrick_elx over 2 years ago
- File superfectar153.jpg added
Look at the attached picture of the way rev 153 displays on Mozilla on Win XP.
The flyby is not working anymore and is always visible and the checkbox has the wrong size.
Updated by tshif over 2 years ago
Wow - it doesnt do that in my firefox. What version are you using? Ill fix this also, but im curious about your version.
Updated by patrick_elx over 2 years ago
what are you trying to do with the <font=2> </font>?
to make the checkbox smaller or the flyby?
In my case, without the font tag, the size of the checkbox is consistent with the rest of the page, and the flyby has the proper size too.
If you want to reduce the size of the checkbox, just move the </font> out of the <a> </a> statement.
Updated by tshif over 2 years ago
- % Done changed from 80 to 90
Niether - goal was to make the text `Test all CID schemes` font size 2 as was done for `Check for Data Source File updates online` text.
Change in font characterstics not possible within class="info" without affecting all occurances of that class. SInce thats not desireable, I will learn to live with larger font than I want for this location. Unless you have a way - :)
Updated by patrick_elx over 2 years ago
- File page.superfecta.php added
same problem with me with your rev 154.
Try the attached file, it's working for me with IE and Firefox.
Updated by tshif over 2 years ago
Ugh! I used the wrong (unfixed) page file for r174. Stupid version control problem that I caused. :(
I cant seem to get a clean copy of your attached file but the problem is obvious - as my r174 still had the errant <font> tags in it.
I need to make some citation changes in the page file anyway - so Ill get a proper version in the SVN very soon.
Sorry about the mistake and extra ticket chatter -
Updated by tshif over 2 years ago
Fixed my mistake by using Patricks code. some citation changes.
Patrick - this works ok for me now - please let me know how it is for you.
Updated by tshif over 2 years ago
- File deleted (
superfectar153.jpg)
Updated by tshif over 2 years ago
- Status changed from QA Testing to Closed
- % Done changed from 90 to 100
QS: Passed cosmetic and functional tests. No negatives noted.
Patrick - if you find problems, we can reopen this case.