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

job type filter in jobs page #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sasikanth513
Copy link

Added simple filter in /jobs page based on the existing one in /profiles page.

Check Below GIF

http://g.recordit.co/62F4BcveYR.gif
1

I want to add more features like

  • Clicking on the label in job post automatically selects label from dropdown and filter the values.

Just want to know you wanted this features.

{{>jobSmall}}
{{/each}}
{{else}}
<p class="text-center">No jobs under this category.</p>
Copy link
Author

Choose a reason for hiding this comment

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

This message might need a look. I'm not native speaker. 😃

@nate-strauser
Copy link
Owner

Thanks! This looks pretty good at first glance. I'll take a deeper look
this weekend.

On Wed, Nov 23, 2016 at 11:51 AM, Sasikanth [email protected]
wrote:

@sasikanth513 commented on this pull request.

In client/views/jobs/jobs.html
#112 (review)
:

  •    <select id="jobType" class="form-control">
    
  •      <option value="All">All</option>
    
  •      {{#each JOB_TYPES}}
    
  •        <option value="{{this}}">{{this}}</option>
    
  •      {{/each}}
    
  •    </select>
    
  •  </div>
    
  • {{#if jobs}}
  • {{#each jobs}}
    
  •     {{>jobSmall}}
    
  • {{/each}}
    
  • {{else}}
  •  <p class="text-center">No jobs under this category.</p>
    

This message might need a look. I'm not native speaker. 😃


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#112 (review),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXyDvkCBWaNQDsoXiO_ttUr7E7FjiHkks5rBG8OgaJpZM4K6z5l
.

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