Skip to content

Gallery button at studio.code.org.#537

Merged
breville merged 5 commits into
stagingfrom
gallerylink
Nov 25, 2014
Merged

Gallery button at studio.code.org.#537
breville merged 5 commits into
stagingfrom
gallerylink

Conversation

@breville

Copy link
Copy Markdown
Member

No description provided.

@breville

Copy link
Copy Markdown
Member Author

@laurelfan how's this? (i moved the hoc link below so that we don't wrap with five items on the line)

@laurelfan

Copy link
Copy Markdown
Contributor

looks good! is there a plan for putting a block in the for the anna/elsa tutorial at some point?

@breville

Copy link
Copy Markdown
Member Author

we will do more blocks for all four new hoc tutorials i think. meeting
about that now.

On Mon, Nov 24, 2014 at 1:23 PM, Laurel notifications@github.com wrote:

looks good! is there a plan for putting a block in the for the anna/elsa
tutorial at some point?


Reply to this email directly or view it on GitHub
#537 (comment)
.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've decided to leave the work for a small box for gallery in this change, so that we have it in the collection if we want to quickly use it somewhere.

@breville

Copy link
Copy Markdown
Member Author

@laurelfan I think this is ready for final review, thanks.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't need this because you don't use caption in this, right?

@laurelfan

Copy link
Copy Markdown
Contributor

lgtm

breville added a commit that referenced this pull request Nov 25, 2014
Gallery bar at studio.code.org.
@breville breville merged commit d903e5b into staging Nov 25, 2014
@breville breville deleted the gallerylink branch November 25, 2014 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants