Closed (fixed)
Project:
Drupal core
Version:
5.x-dev
Component:
base system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
26 Aug 2006 at 14:13 UTC
Updated:
13 Mar 2008 at 10:41 UTC
Jump to comment: Most recent file
New patch for proper unit-usage in core.
Here is what the patch does:
- it makes use of drupal api's to display the part of the url in front of path textfields (path.module/system.module);
- it gives us consistent textfield sizes between upload/user module for specifying resolutions and file sizes;
- have user.module make use of the '#field_(prefix|suffix)' where needed;
- did some small part of more consistent code markup for the appropriatary functions;
Please review and apply...
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | example.png | 216.23 KB | Stefan Nagtegaal |
| unit-usage-for-forms_2.diff | 10.01 KB | Stefan Nagtegaal |
Comments
Comment #1
Stefan Nagtegaal commentedHere is some screenshot which shows the difference with this patch applied.
Drupal now *could* show the base_url in front of the path specification field, using the '#field_prefix'..
Much better IMO..
thoughts? reviews?
Comment #2
AjK commentedNice idea but doesn't seem to be a "bug" more a feature request.
Re-categorising.
regards,
--AjK
Comment #3
flk commentedi would have to agree with ajk, this is more inline with a feature request than a bug. just my 2cents
Comment #4
Stefan Nagtegaal commentedSince the $form['prefix'] and $form['suffix'] i in core now, this is a bug report because we do not use them like we should..
Though I have to admit that it stinks a little to get a new feature in, but let our core committers decide on that. ;-)
Comment #5
dries commentedI looks uglier to me, but it might be easier to use. Not sure.
Comment #6
drummCommitted everything except the user.module part which no longer applied and should probably have been posted separately anyway.
Comment #7
ricabrantes commented4.x is no longer supported.. This bug are fixed in D5.x-dev
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.