forked from erector/erector
-
Notifications
You must be signed in to change notification settings - Fork 1
/
todo.txt
51 lines (36 loc) · 1.24 KB
/
todo.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
use rvm to test locally under 1.8.6
rename to_s to to_html
change instance vars to start with @_
rename 'user guide' to 'documentation' or something more clear
rename "external" to "depends_on"
variant syntaxes for "depends_on"
depends_on :js => ["foo.js", "bar.js"]
depends_on FooWidget, BarWidget
-- add all of these widgets' dependencies
cleverly figure out the external type
depends_on "foo.js"
depends_on "foo.css"
depends_on File.new("foo.js")
depends_on File.new("foo.css")
unify :script and :js, :style and :css
make render switch inside ExternalRenderer
class FooWidget < Widget
depends_on "jquery.js", :render => :link
depends_on "foo.js"
depends_on "bar.js"
end
ExternalRenderer.default_render = :embed
pluggable renderer e.g.
class JavaScriptRenderer < Widget
def content
if @external.options[:render] == :embed
javascript @external.options do
rawtext @external.text
end
else
script({:type => "text/javascript", :src => @external.text}.merge(@external.options))
end
end
end
ExternalRenderer.renderer_for(:js, JavaScriptRenderer)
rename 'css' convenience method to 'css_link' method, or otherwise remove confusion between inline and linked css