Skip to content
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.

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

Footer Styles #17

Open
ryuhhnn opened this issue Nov 24, 2016 · 20 comments
Open

Footer Styles #17

ryuhhnn opened this issue Nov 24, 2016 · 20 comments
Assignees
Milestone

Comments

@ryuhhnn
Copy link
Collaborator

ryuhhnn commented Nov 24, 2016

Make the footer area look a little more clean and consider adding in a few different columns.

@ryuhhnn ryuhhnn assigned ryuhhnn and unassigned ryuhhnn Nov 24, 2016
@frankinedinburgh
Copy link

I can try do a bit of work on this.

@Pakistanka
Copy link

Hello. I can try to help if it needs

@ryuhhnn
Copy link
Collaborator Author

ryuhhnn commented Feb 9, 2017

@Pakistanka That would be awesome! Submit a PR when you're ready :)

@bilyanhadzhi
Copy link

Hello, @ryuhhnn! Are you accepting contributions? I've noticed you don't include a meta viewport tag in base.jade, leading to non-responsive pages.

I see great potential to contribute to this project. I don't know node, express, jade too well but I guess they should be easy to pick up.

@agauniyal
Copy link
Owner

@bilyanhadzhi Sure. Feel free to open Pull Requests against develop branch.

@ollyjohn
Copy link

Is this still needing help? I'd be more than happy to lend a hand.

@agauniyal
Copy link
Owner

@ollyjohn Yep, till the issues aren't closed or a corresponding PR isn't open, feel free to work on them.

@bilyanhadzhi
Copy link

Is there another place for discussing the project, so we don't populate just this issue?

Anyway, I skimmed over the code and have a few suggestions:

  • Adding a meta viewport tag in base.jade is a must
  • Prevening the submit button from submitting the form:
    • It could be left as is, and expect the user to add return false to the form tag
    • We could have a class like nosubmit/no-submit/form-nosubmit which (via js) changes the onsubmit property of any elements that has it to return false
  • For the footer itself, we could have CSS classes such as big, medium and small which would be added to the footer element and based on them we could control how big the 'footer area' is.
    • For example, for footer.big we'd have a bigger area, like this, which seems to be how big most footers are.

I'm willing to open issues for the things suggested (and work on this one as well).
I'm writing this so as to see if you agree with my suggestions about the footer, so I could implement them that way.

@ryuhhnn
Copy link
Collaborator Author

ryuhhnn commented Feb 17, 2017

@bilyanhadzhi there isn't any place for discussion on this project currently, unless @agauniyal has implemented any? Outside of that, you're more than welcome to submit individual issues on these matters and keep related discussion within those!

@bilyanhadzhi
Copy link

I'm having trouble getting my SCSS to take effect. I created a new file is /sass/ and imported it to main.scss, gulp doesn't seem to be noticing. Any help?

@ryuhhnn
Copy link
Collaborator Author

ryuhhnn commented Feb 17, 2017

@bilyanhadzhi Could you maybe fork over the repository and provide some steps to recreate?

@bilyanhadzhi
Copy link

bilyanhadzhi commented Feb 17, 2017

@ryuhhnn Sure. What I did was:

  1. Fork and clone the repo
  2. $ git checkout develop
  3. $ git branch footer-work && git checkout footer-work
  4. $ npm install
  5. $ touch sass/_footer.scss
  6. Write styles for a class in _footer.scss and @import 'footer' in main.scss
  7. $ gulp
  8. Add the class to an element in base.jade - element gets the class but no styles take effect

gulp also does not really do anything when I save any .scss file.

I could open a pull request with the code I have so we can see what I'm doing wrong? It's just one basic style in _footer.scss

@ryuhhnn
Copy link
Collaborator Author

ryuhhnn commented Feb 17, 2017

@bilyanhadzhi could you please push changes to your forked repo? It would be better for me to take a look at it that way.

@bilyanhadzhi
Copy link

bilyanhadzhi commented Feb 17, 2017

@ryuhhnn Just did. Also I see public/css/main.css was deleted after running some gulp task.

@agauniyal
Copy link
Owner

@agauniyal
Copy link
Owner

agauniyal commented Feb 18, 2017

@bilyanhadzhi gulp tasks might be in a mess atm, I haven't even seen develop branch for a while since I am busy with finishing my course. When I started this project I wanted to keep it minimal, that's why I picked only sass as dependency and kept out other things. Along the way some nice contributors added very good features which enhanced the project and added some extra things, which in my opinion aren't necessary. For example, we aren't really using server for doing anything except provide a short localhost:port url, which is already provided by browsersync gulp plugin.

So I'm thinking of working on this project again and restructure it well while keeping all essential things like styles and stuff and removing extraneous gulp plugins and server. I am also thinking of shifting to component based framework where user could do something like -

<html>
<head> ...</head>
<body>
<hero-image></hero-image>
<wide-slider></wide-slider>
<about-us>
    <p></p>
    <portfolio></portfolio>
</about-us>
<wide-footer></wide-footer>
...
...

which will be very nice for end user.

@ryuhhnn ^ your thoughts?

@bilyanhadzhi
Copy link

Yeah, I figured a rewrite may be the most appropriate thing to do. There are many deprecated node packages being used too since it has not been maintained for some time.
Overall, developing in its current state seems impossible to me.

@agauniyal agauniyal added this to the 1.0.0 milestone Jul 24, 2017
@erickrod54
Copy link

Hello @agauniyal , is still open the issue ? i would like to work to do it :), can you assign it to me ?

@Chetax
Copy link

Chetax commented Mar 11, 2023

Hello @agauniyal , is still open the issue ?

@t-li-projects
Copy link

Hello, if the issue is still open, i'd like to fix this issue, can you please assign me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants