LuckyFlow::Element#value uses property instead of attribute

[matthewmcgarvey]

The reasoning behind this was laid out here: #86

TLDR: switching to property allows us to add support for other browsers in the future

Fixes #86

[matthewmcgarvey]

while attribute is no longer used in the class, I wasn't sure about removing it from this list since it is technically public API. I can remove it, though

[jwoertink]

I think we should keep it in. At least for this next release. We can see about deprecating it later.

[jwoertink]

I think this is fine. Can we have a spec that shows using this just so we can verify it works?

[matthewmcgarvey]

@jwoertink Do you mean a spec testing the value method? If so, here it is

lucky_flow/spec/lucky_flow_spec.cr

Lines 77 to 90 in 78f895d

	it "can fill in text" do
	flow = visit_page_with <<-HTML
	<input name="question:title"/>
	<input name="question:body"/>
	HTML
	flow.field("question:title").value.should eq ""
	flow.field("question:body").value.should eq ""
	flow.field("question:title").fill("Joe")
	flow.fill "question:body", with: "Sally"
	flow.field("question:title").value.should eq "Joe"
	flow.field("question:body").value.should eq "Sally"
	end

[jwoertink]

I was thinking more on the property method itself, but now I think value being called is probably good enough. 👍