View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0002219 | NetSurf | RISC OS-specific | public | 2014-11-13 12:07 | 2016-02-16 15:25 | ||||
Reporter | Sprow | ||||||||
Assigned To | Steve Fryatt | ||||||||
Severity | minor | Reproducibility | always | ||||||
Status | closed | Resolution | no change required | ||||||
Product Version | 3.2 | ||||||||
Target Version | 3.4 | Fixed in Version | 3.4 | ||||||
Summary | 0002219: Sprite ptr_lr placed into the main Wimp pool | ||||||||
Description | File '!NetSurf.5Sprites22' and '!NetSurf.!Sprites22' both contain "ptr_lr", this also appears in !NetSurf.Resources.Sprites in your private pool, so the other 2 should be deleted. | ||||||||
Tags | No tags attached. | ||||||||
Fixed in CI build # | |||||||||
Reported in CI build # | |||||||||
URL of problem page | |||||||||
Attached Files |
|
Notes | |
Sprow (reporter) 2015-08-16 08:40 Last edited: 2015-08-16 08:41 |
Having looked at the reason for this in the code, I see that it's to support the widget in the status bar that sets where the split line is. That uses 'P' validation on the icon, which in turn requires the sprite to be in the Wimp pool. The other uses of the ptr_ sprites are elsewhere in the browser where the private pool is appropriate. While it would in theory be possible to swap the widget to button type 1 ("always") and replicate the Wimp's logic, this seems a waste of time. Instead, I've requested (and been granted) that ptr_lr be allocated with ROOL so it's now reserved for NetSurf use. Can close this ticket now. |
John-Mark Bell (administrator) 2015-08-16 23:52 |
Resolved. Thanks! |
Vincent Sanders (administrator) 2016-02-16 15:25 |
Confirmed resolved in 3.4 release |
Issue History | |||
Date Modified | Username | Field | Change |
---|---|---|---|
2014-11-13 12:07 | Sprow | New Issue | |
2014-11-13 16:49 | Vincent Sanders | Assigned To | => Steve Fryatt |
2014-11-13 16:49 | Vincent Sanders | Status | new => acknowledged |
2015-08-16 08:40 | Sprow | Note Added: 0000917 | |
2015-08-16 08:41 | Sprow | Note Edited: 0000917 | View Revisions |
2015-08-16 23:52 | John-Mark Bell | Note Added: 0000918 | |
2015-08-16 23:52 | John-Mark Bell | Status | acknowledged => resolved |
2015-08-16 23:52 | John-Mark Bell | Resolution | open => no change required |
2015-08-16 23:52 | John-Mark Bell | Fixed in Version | => 3.4 |
2015-08-16 23:52 | John-Mark Bell | Target Version | => 3.4 |
2016-02-16 15:25 | Vincent Sanders | Note Added: 0001299 | |
2016-02-16 15:25 | Vincent Sanders | Status | resolved => closed |