Skip to content
This repository has been archived by the owner on Aug 9, 2018. It is now read-only.

Treelist Stylesheet #93

Open
wylie opened this issue Apr 9, 2015 · 4 comments
Open

Treelist Stylesheet #93

wylie opened this issue Apr 9, 2015 · 4 comments

Comments

@wylie
Copy link

wylie commented Apr 9, 2015

This stylesheet, less/_treelist.less, needs some cleanup. Remove the foo: bar and commented out lines.

@wylie wylie self-assigned this Apr 9, 2015
@wylie wylie added this to the Reassess Widgets milestone Apr 9, 2015
@wylie
Copy link
Author

wylie commented Apr 15, 2015

I'm thinking that the treelist needs to be seriously worked on. Our stylesheet currently has this structure

.treelist {
    .list {}
}

I think that we should combine the two and make the treelist an actual list. The .treelist class actually has no styles tied to it, so this could be a simple task of combining the classes (.list.treelist as opposed to .treelist .list) and revising the template.

Then we could remove this stylesheet, by merging it into the lists stylesheet, or keep it as a way to differentiate the different types of lists we use.

For now I have removed some commented out declarations and the foo: bar; business.

@gjjones
Copy link
Contributor

gjjones commented Oct 27, 2015

Is a treelist something we'd even want in edna? My first thought is to have edna cognizant of single layer lists, and have nests lists play friendly. But to explicitly state that UI pattern is a tree list makes me think more of a buzz widget than an edna component.

@wylie
Copy link
Author

wylie commented Oct 27, 2015

yep, this should live in Buzz. Once we go atomic it definitely will, but the base list styles will be in Edna.

@gjjones
Copy link
Contributor

gjjones commented Oct 27, 2015

👍

@wylie wylie removed their assignment Jul 31, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants