Skip to content
This repository has been archived by the owner on Dec 23, 2017. It is now read-only.

Feature/panel desktop #392

Merged
merged 4 commits into from
Aug 3, 2015
Merged

Conversation

msecret
Copy link
Contributor

@msecret msecret commented Aug 2, 2015

Check commit messages for explanations

msecret added 2 commits August 1, 2015 21:41
This is the basics for the panel interface on above mobile. This
opens a panel to the right side of the table at 50%. This might be
hard to animate, we might want to look at that later.

TODO:
- Create arrow icon on right side of each row to open the panel.
- Change close button and icon for this breakpoint.
- Figure out panel correct heading.
- Figure out panel indentation.
The headers are supposed to be smaller on larger screens. There was
an indentation problem with the fields of the panel. This was due
to them being a td under dataTable, which in turn uses an !important
rule likely because it has to override dataTable's inline code.

This all has to be fixed but would likely be a larger refactor,
so am ignoring now with a shameful rule.
@noahmanger
Copy link
Contributor

This is looking really good. Not sure if you're still working on it, but the only thing I see worth noting is that the active row should be highlighted.

@msecret
Copy link
Contributor Author

msecret commented Aug 3, 2015

Yea I was still working on a few things like that. I also wanted to get the arrow on the rows done, move the close button on desktop.

What the status of those "View original image" button? Is there an existing link for that?

Also, for the perma-link, I'm guessing it will take about a day to implement that. Is it something I should do this next sprint, or are there other things that are more important?

@noahmanger
Copy link
Contributor

Yeah I'd say it'd be good to do that this sprint. The "original image" link should exist...it may need to be built using the image numbers. @LindsayYoung how are we doing this now?

@noahmanger
Copy link
Contributor

Regardless, I think we can merge this in whenever the active state is ready.

}

#datatable-modal .panel .panel__row {
// TODO this shit is broken.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's broken?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha, I was supposed to go back to that and write a better comment, I'll do so now

noahmanger pushed a commit that referenced this pull request Aug 3, 2015
@noahmanger noahmanger merged commit 1ba4f1f into fecgov:develop Aug 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants