[WIP] Fix/image link

[ryan-berger]

Fixes #312

[codecov-commenter]

Codecov Report

Merging #322 into master will decrease coverage by 1.04%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #322      +/-   ##
==========================================
- Coverage   81.04%   80.00%   -1.05%     
==========================================
  Files           8        8              
  Lines         902      915      +13     
==========================================
+ Hits          731      732       +1     
- Misses        171      183      +12     

Impacted Files	Coverage Δ
lib/src/interactable_element.dart	50.00% <14.28%> (-50.00%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a8e58b...d4b3958. Read the comment docs.

[andy1xx8]

Too many abstraction levels in the body of 1 method.
You should break it down to smaller method/func

Infact, the code should do like this ( i will assume that your code logic is correct)

if(hasNoChild(interactableElement)) {
underline(interactableElement);
} else {
underlineChildTextElements(interactableElement);
}

Then, the method underlineChildTextElements will take care the recurrsive search & apply the corresponding style to its children

[andy1xx8]

Too many abstraction levels in the body of 1 method.
You should break it down to smaller method/func

Infact, the code should do like this ( i will assume that your code logic is correct)

if(hasNoChild(interactableElement)) {
underline(interactableElement);
} else {
underlineChildTextElements(interactableElement);
}

Then, the method underlineChildTextElements will take care the recurrsive search & apply the corresponding style to its children

[erickok]

Alternative fix: #499 499

[ryan-berger]

Closing due to #499