Skip to content

Ep site section#104

Closed
jadefaerie wants to merge 2 commits into
entry-pagefrom
ep-site-section
Closed

Ep site section#104
jadefaerie wants to merge 2 commits into
entry-pagefrom
ep-site-section

Conversation

@jadefaerie

Copy link
Copy Markdown
Contributor

This checkin is for the site-section ticket from Trello (https://trello.com/c/QQ5QNzSE/1-site-section-c). Added incomplete arrow mixin that will be used in this component. Colors and final finessing is needed for the site section styles.


&.mentorship {
//mentorship-specific styles here
background-color: blue; // temp color

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.

These comments and temporary colors are fine for now. We'll finesse when everything is merged into the entry-page branch.

}

//use this for larger screen sizes, define the width of the screen in _variables.scss
@media (min-width: $width-lg) {

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.

Looks like this isn't being used. I'll nuke it after the merge.

}
}

//testing only -- this is not the "real" featured-list

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.

This should be nuked, too.

<a href="#">arrow goes here</a>
</div>

<!--featured list component goes here -->

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.

Thanks for the comment here. :)

@@ -1,5 +1,49 @@
<%# render "layouts/navbar", page: "" %>
<!--base section -->

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.

I'll clean this up after merge so you don't have to mess with it.

@nickirios nickirios left a comment

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.

Nuke the ruby changes, and this'll be good to go. :)

@@ -0,0 +1,16 @@
require "rails_helper"

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.

I don't remember modifying any ruby. I don't think this should be here. Can you remove this file from the PR?

Comment thread spec/rails_helper.rb
@@ -0,0 +1,77 @@
# This file is copied to spec/ when you run 'rails generate rspec:install'

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.

Same here. Remove this file for the PR, please.

Comment thread spec/spec_helper.rb
@@ -0,0 +1,99 @@
# This file was generated by the `rails generate rspec:install` command. Conventionally, all

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.

Remove this one from the PR, too.

@@ -0,0 +1,43 @@
require 'spec_helper'

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.

Remove this one from the PR, too.

@natsteinmetz natsteinmetz deleted the ep-site-section branch August 4, 2018 22:25
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.

3 participants