[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[EP-tech] Re: Advanced Search Page render of large number of choices (such as language).



Adam, Gilles,

Thank you both for your replies.  I've gone ahead and changed the search_input_style to 'default' instead of 'select' and got the same rendering on the advanced search - perfect.  So now we're at least "safe" going forward.

Thanks again,
Brian Gregg.


-----Original Message-----
From: eprints-tech-bounces at ecs.soton.ac.uk [mailto:eprints-tech-bounces at ecs.soton.ac.uk] On Behalf Of Field A.N.
Sent: Tuesday, September 01, 2015 10:27 AM
To: eprints-tech at ecs.soton.ac.uk
Subject: [EP-tech] Re: Advanced Search Page render of large number of choices (such as language).

I would recommend against 'select' because it implies that you've got a select box because of the word you used.  Also, in the future if 'select' ever does become a valid option, you may be surprised by unexpected behaviour.  I suggested 'default' because it's actually already being used:

	https://github.com/eprints/eprints/search?utf8=%E2%9C%93&q=search_input_style

--
Adam Field
Business Relationship Manager and Community Lead EPrints Services
+44 (0)23 8059 8814





On 1 Sep 2015, at 15:23, Gilles Fourni? wrote:

> Hi,
> 
>> I've made the suggested changes.  However, I see in my advanced search languages field is defined as '{ meta_fields => [ "documents.language" ] },'  which is not defined in either of the eprint_fields.pl or document_fields.pl in the archive config directory.  But I did change our 'degree' field in the eprint_fields.pl to use 'search_input_style' => 'select' (Thanks for the tip Gilles.) and it is now showing as a select list box.  So I now see that using the 'search_input_style' change does indeed work.  Is there some indication anywhere of what the available options are for this as this is the first time I've run across this?
> 
> From the code, it seems that the only value checked is "checkbox".
> 
> So, I think that we can only use either "checkbox" or anything else.
> I have chosen to use "select" but, as Adam wrote, the value "default" 
> should do the same thing.
> 
> Gilles
> *** Options: 
> http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
> *** Archive: http://www.eprints.org/tech.php/
> *** EPrints community wiki: http://wiki.eprints.org/
> *** EPrints developers Forum: http://forum.eprints.org/


*** Options: http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
*** Archive: http://www.eprints.org/tech.php/
*** EPrints community wiki: http://wiki.eprints.org/
*** EPrints developers Forum: http://forum.eprints.org/