Opened 14 years ago
Closed 14 years ago
#4276 closed enhancement (duplicate)
height/width cleanup
Reported by: | mike.helgeson | Owned by: | brandon |
---|---|---|---|
Priority: | minor | Milestone: | 1.4 |
Component: | dimensions | Version: | 1.3.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
The changes recently made to improve the height/width methods left behind some unused code, and some duplicate code. The attached file is from r6195...
I removed the unused "tl" and "br" variables.
I removed the "type" variable (which is exactly the same as "lower"), and replaced all usage of "type" with "lower."
I added a property lookup to the "height" and "width" methods. Currently if the "size" argument is undefined, the method is a getter. With this change, as a getter, the argument can be "padding" or "border" or "margin" or undefined. This allows the same behavior as achieved by the "inner..." and "outer..." methods.
Attachments (1)
Change History (3)
Changed 14 years ago by
Attachment: | dimension_r6195.js added |
---|
comment:1 Changed 14 years ago by
Component: | core → dimensions |
---|---|
Owner: | set to brandon |
comment:2 Changed 14 years ago by
Resolution: | → duplicate |
---|---|
Status: | new → closed |
The dimensions module has since been cleaned up.