-
Notifications
You must be signed in to change notification settings - Fork 2
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
New Tabs and Language Options for CBSS Browse page #184
Comments
Please make the following changes to https://dev.syriaca.org/cbss/browse.html
Please order the tabs horizontally in this order with English on the far left and Syriac on the far right: A-Z (I list them vertically here in the issue just so the order is clear, the github text box is freaking out about the list having multiple right to left languages in it). Next, there are some issues with how the browse results are being filtered or sorted. It looks like the current alphabetical filter is running on the first name rather than the last name. Can we fix that please? Please use: Next it looks like the sorting by language is not working as expected. I will create a separate issue for that. |
Wow, now the RTL issue won't let me edit the box at all :) So I am completing my thoughts from above here. Next, there are some issues with how the browse results are being filtered or sorted. It looks like the current alphabetical filter is running on the first name rather than the last name. Can we fix that please? Please use: /TEI/text/body/biblStruct/analytic/author/surname (first letter of surname text string) Next it looks like the sorting by language is not working as expected. I will create a separate issue for that. |
just an fyi on the name sorting, not all the records will have forename and surname; e.g., https://dev.syriaca.org/cbss/35F77WHT. So I think prefer sort by tei:surname, but backup would be sort by tei:name |
@wlpotter @davidamichelson Did we make any decisions about the browse page? I seem to remember the default being browse by author? Do we need the different language tabs? Anything else?
The text was updated successfully, but these errors were encountered: