wireframe icon indicating copy to clipboard operation
wireframe copied to clipboard

Footer Styles

Open ryuhhnn opened this issue 8 years ago • 20 comments

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

ryuhhnn avatar Nov 24 '16 00:11 ryuhhnn

I can try do a bit of work on this.

frankinedinburgh avatar Nov 24 '16 21:11 frankinedinburgh

Hello. I can try to help if it needs

Pakistanka avatar Feb 09 '17 20:02 Pakistanka

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

ryuhhnn avatar Feb 09 '17 21:02 ryuhhnn

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.

bilyanhadzhi avatar Feb 15 '17 23:02 bilyanhadzhi

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

agauniyal avatar Feb 16 '17 05:02 agauniyal

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

ollyjohn avatar Feb 16 '17 20:02 ollyjohn

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

agauniyal avatar Feb 17 '17 06:02 agauniyal

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.

bilyanhadzhi avatar Feb 17 '17 19:02 bilyanhadzhi

@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!

ryuhhnn avatar Feb 17 '17 19:02 ryuhhnn

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?

bilyanhadzhi avatar Feb 17 '17 20:02 bilyanhadzhi

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

ryuhhnn avatar Feb 17 '17 21:02 ryuhhnn

@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

bilyanhadzhi avatar Feb 17 '17 22:02 bilyanhadzhi

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

ryuhhnn avatar Feb 17 '17 23:02 ryuhhnn

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

bilyanhadzhi avatar Feb 17 '17 23:02 bilyanhadzhi

@BilyanHadzhi @ryuhhnn https://gitter.im/discuss-wireframe/

agauniyal avatar Feb 18 '17 07:02 agauniyal

@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?

agauniyal avatar Feb 18 '17 07:02 agauniyal

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.

bilyanhadzhi avatar Feb 18 '17 08:02 bilyanhadzhi

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

erickrod54 avatar Sep 23 '22 03:09 erickrod54

Hello @agauniyal , is still open the issue ?

Chetax avatar Mar 11 '23 02:03 Chetax

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

t-li-projects avatar Aug 09 '23 09:08 t-li-projects