Skip to content

Filter arrange #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Nava2 merged 5 commits into from
Apr 17, 2015
Merged

Filter arrange #30

Nava2 merged 5 commits into from
Apr 17, 2015

Conversation

@Nava2
Copy link
Contributor

@Nava2 Nava2 commented Apr 16, 2015

  • Adds models
  • Move filters as a column instead of atop.

screen shot 2015-04-16 at 19 00 38

@aforward
Copy link
Contributor

aforward commented Apr 17, 2015

Not in love with the side look. What about a hideable / expandable one on top. Go ahead and merge if you think it's better

@Nava2
Copy link
Contributor Author

Nava2 commented Apr 17, 2015

I don't have great feelings either way, I do prefer the side one though. Maybe @umple could chime in? It scrolls with the page (which @aforward says is worse 😛)

@TimLethbridge
Copy link
Member

TimLethbridge commented Apr 17, 2015

I would have to actually use the side view. I am OK with merging. We could perhaps experiment with it for a bit.

@Nava2
Copy link
Contributor Author

Nava2 commented Apr 17, 2015

@aforward and I talked a bit offline. We were thinking of using a single "search" bar style input, a la google search with suggested keywords.

@Nava2 Nava2 mentioned this pull request Apr 17, 2015
Open
@Nava2
Copy link
Contributor Author

Nava2 commented Apr 17, 2015

I'm going to merge this. I'll investigate "omnibar" search within #32.

Nava2 added a commit that referenced this pull request Apr 17, 2015
@Nava2
Merge pull request #30 from umple-ucosp/FilterArrange
5fe873f 
Filter arrange
@Nava2 Nava2 merged commit 5fe873f into master Apr 17, 2015
@Nava2 Nava2 mentioned this pull request Apr 17, 2015
Closed
@Nava2 Nava2 deleted the FilterArrange branch April 18, 2015 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Nava2 @aforward @TimLethbridge