First things first... I got it to validate.
Was a little panicky when I saw over 200 errors... Amazing the things we forget as we're coding eh?
Ok, on to MTO's comments:
[]
MTO said:Wooow! Looking great! I'm liking what I see Anyway... I'm always looking for ways to improve things, so here are my other suggestions & observations.
[/]
Thanks, and I agree, I can see a bunch of ways to make this better. It's good to have others give input as well.
[]
Edit Profile Screen:- Just template cosmetic fixes: "Geographic Location (State, Country, etc.) " should be right after "Location Country" or it will seem repetitive to most people. "Location Country" should probably be before "city".
[/]
Actually, that's straight out of the CMS mod.
When I release this the prerequisites will be the CMS mod and the birthday mod. Although... I could probably wrap it into a single add on.
I'll have to ask for permission first if I decide to do that. I did notice some errors in documentation in the two mods.
[]
- Maybe we could have a "Be available in profile searches? Yes/No"
[/]
I agree. I didn't do this right off as I was trying to steer clear of database modifications. But an extra field can't hurt.
[]
Search Members Page:- When you first enter, it shows results. I havent searched for anything yet... Results should be empty. Is there a limit to the number of results per page? Or will it show all 3000 members in the same page if you had them?
[/]
Right, I have the fields set to "empty" on load, mostly so I could quickly see changes as I ran the first queries and I knew how many members SHOULD show up, if all fields are empty.
I could get it to just show the form on load, sure.
[]
- How about a "Text in bio" search field?
[/]
Can do that. Shouldn't be too hard. Tons easier than that darned age thing. Just wait until you see THAT code.
[]
- "Name" search field. Should search within "Display name" and "Full name".
[/]
Hmm, true, I didn't know which field I should use. Only a few members of my board use the Display Name field and the Full Name field was populated, I THINK when I ported from .Classic. This should also be easy to add. Just another part of the query.
[]
- "Country" search field. Maybe should search within both location & origin country?
[/]
Also debated on this... Perhaps have both available? Again, very easy, just one more AND to the changeable query I built.
[]
- "Has a picture" field. Maybe too much to ask for...
But it would be nice if an avatar wouldnt be considered a picture and users could actually really add a picture to their profile since these are usually not pictures (though mine is).
[/]
Good point. Avatars on my site are taken from screenshots in the games we play on. Avatar and Picture are one in the same right now. So that's the field I'm searching on... we could add another field to the DB that is a "true" picture field and add it to the showprofile template.
[]
Search results- I think it would be really nice if it would display some information about them. Maybe show the avatar, Name, Full name, number of posts, age, country, and the first 15 words or so of their bio.
- X results per page.
[/]
I want to do this. Right now I didn't know what people would want to see there. Avatars in the list could make the page quite long and cumbersome. Then again, some table tweaking could fix that.
As far as having x number of results per page. I know that this would be needed, because I expect some of the sites that use this would have WAY more than my 350 users. Talk about a long page with 3000+ results.
[]
Anyway... Great job!
It's looking great. [/]
Again thanks. And you and I are thinking along the same wave length regarding the changes. Think I mentioned before that I had other ideas mapped out in my head already. I dreamt about coding PHP all last night.
Will sleep on it and start making more changes tomorrow.