smstw / windwalker-joomla-rad

Windwalker RAD framework for Joomla CMS
http://ventoviro.github.io/windwalker-rad-doc/
1 stars 2 forks source link

[Update #35]add htmlBuilderTest #36

Closed skylying closed 9 years ago

skylying commented 9 years ago

For #35

asika32764 commented 9 years ago

那個 attribsattributes 的常用簡寫

skylying commented 9 years ago

我先改成 DataProvider 的形式, 麻煩大大幫我看一下這樣結構可不可以摟, 至於要怎樣 assert 我再稍微研究一下

asika32764 commented 9 years ago

大哥,不需要用到 DOMDocument 啦......

這樣會造成可讀性變非常差

asika32764 commented 9 years ago

這兩個請參考 https://github.com/ventoviro/windwalker/blob/staging/src/Dom/Test/HtmlElementTest.php

https://github.com/ventoviro/windwalker/blob/staging/src/Dom/Test/HtmlBuilderTest.php

skylying commented 9 years ago

我構思這麼久你居然直接打我槍 我哭了

skylying commented 9 years ago

為什麼你有 DOMHelper 可以用, 犯規啦

skylying commented 9 years ago

大致上都改好囉

skylying commented 9 years ago

ok, 完成 HtmlBuilder 改好了, 現在可以支援 <video controls muted></video> 格式囉

asika32764 commented 9 years ago

:+1:

skylying commented 9 years ago

我發現另一個問題誒.... 如果我 create 這樣送參數的話

HtmlBuilder::create('img', 'Hello world');

他會回傳

<img> 
    Hello world
</img>

這是不是要擋一下?, 如果 屬於 unpaired tag 就忽略 $content 參數

p.s 其實我是在寫 HtmlElementTest 的時候發現的, 因為他裡面直接調用 HtmlBuilder::create();, 所以有時候 content 在外面會先放進去

asika32764 commented 9 years ago

我本來的打算是個人造業個人擔,自己要送 Content 進去的就強制變成成對 tags

要擋也是可以的,不過我建議等 2.1 開始正式開發時再討論,到時候我要把 framework 上的 Html package 跟這邊整合在一起

skylying commented 9 years ago

那我這隻 PR 沒有要改的了, 請您融合吧