-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add commander search #12
Comments
I would totally love this feature. Keep up the good work. |
Any progress on this? As I say whenever I come around here ... coding is not my thing, so feel free to tell me to fuck off ... but this feature I think would be awesome ... just to see where old friends/enemies have moved on to. |
I don't need help with coding this per se. But I'm having a hard time figuring out where to add the search function in the site. |
Maybe something like this?
http://bootsnipp.com/snippets/GzXXl
Get Outlook for iOS<https://aka.ms/o0ukef>
On Thu, Dec 29, 2016 at 10:03 AM -0500, "Kender2" <notifications@github.com<mailto:notifications@github.com>> wrote:
I don't need help with coding this per se. But I'm having a hard time figuring out where to add the search function in the site.
For the desktop version it's easy enough to put a link, button or searchbox somewhere, but since most users are on mobiles (62% mobile, 18% tablet, 20% desktop) I prefer an elegant and intelligent solution.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<#12 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AWERbdbjiOakJXwdz5b1GCCD7GcDRhfxks5rM8vYgaJpZM4LSH0_>.
|
oh snap, should have checked out the issue before coding the PR. Anyway, snippet with dropdown looks nice, should be easy to integrate. |
…arch, search form fixes
Done |
Add functionality to find a player through search.
Results show a list of matching commanders and their squad and faction.
Question is where to place the search box or link to search page.
The text was updated successfully, but these errors were encountered: