vivo-project / VIVO

VIVO is an extensible semantic web application for research discovery and showcasing scholarly work
http://vivoweb.org
BSD 3-Clause "New" or "Revised" License
205 stars 127 forks source link

I18n redesign #3805

Closed chenejac closed 1 year ago

chenejac commented 1 year ago

VIVO GitHub issue: i18n redesign project board

Vitro PR

What does this pull request do?

Moving UI labels from property files to rdfs Removing language specific template files Merging VIVO-languages and Vitro-languages repositories into VIVO and Vitro repositories Support for institution specific translations (private language subtags)

What's new?

.property UI label files are migrated to .ttl files i18n module is extended to support loading of UI labels from ttl files and property files ftl files from VIVO-language have been moved to VIVO repository (universal language constructions working for all languages has been used) home artifacts and UI label files from VIVO-languages have been moved to VIVO pom files have been adopted *.yml files (build) have been adopted

How should this be tested?

Example:

Interested parties

Tag (@ mention) interested parties or, if unsure, @VIVO-project/vivo-committers

ghost commented 1 year ago

Some languages has undesired word wrapping.

image

Can't find Journal of Science in sample data.

image

What context is there does render in all available languages.

image

French languages missing ellipses if they are intended to use. Also, the bottom horizontal line is not bottom padded as much as the other like sections.

image

When adding advisee typeahead selection of existing individual not working as expected.

image

Otherwise, worked as final line item stated.

image

Padding is cramped. Worse in some languages. But all have no space right of the search label.

image

Also, both French Canadian is showing blank page.

image

Russian locale (ru_RU) is missing create password link in email template.

image

image

Unable to find matches to labels suffixed with uqam. Possibly, the setup process is incomplete. See https://github.com/TAMULib/VIVO/blob/i18n-redesign-docker/start.sh#L39 in attempts to automate setup instructions.

image

image

Range type selected was XML literal and it failed to create faux data property.

image

image

Does allow create faux data property for range type string.

image

chenejac commented 1 year ago

Unable to find matches to labels suffixed with uqam. Possibly, the setup process is incomplete. See https://github.com/TAMULib/VIVO/blob/i18n-redesign-docker/start.sh#L39 in attempts to automate setup instructions.

image

Yes, I think automate setup is not configured well for the example of uqam UI labels localization. It is not in sample-data repository, it can be only downloaded from this link at the moment - https://github.com/vivo-project/Vitro/files/10185798/fr_CA_x_uqam.zip. So, the changes are not in sample data, the changes are in labels translations.

ghost commented 1 year ago

@chenejac I followed step

As can be seen here

https://github.com/TAMULib/VIVO/blob/i18n-redesign-docker/start.sh#L43

from

https://github.com/wwelling/sample-data/tree/main/fr_CA_x_uqam

It looks as I need to only copy a directory within the zip file.

https://github.com/wwelling/sample-data/tree/main/fr_CA_x_uqam/interface-i18n/firsttime into $VIVO_HOME/rdf/i18n/

I will give this a try.

ghost commented 1 year ago

@chenejac this looks to be what is needed

cp -r sample-data/fr_CA_x_uqam /usr/local/vivo/home/rdf/i18n/fr_CA_x_uqam

currently redeploying now and will test

This works as expected with putting sample data in correct place.

image

@litvinovg I am still receiving this error page when trying to create a faux property from an XML literal.

The image is and was using the latest from upstream i18n-redesign. See https://github.com/TAMULib/VIVO/blob/i18n-redesign-docker/Dockerfile#L30

image

Here is the screen before the error page.

image

leaving only this test remaining

ghost commented 1 year ago

image

labs.library.tamu.edu/:264 Uncaught SyntaxError: Unexpected identifier 'Université'

?uri=http%3A%2F%2Flocalhost%3A8080%2Fvivo_i18n%2Findividual%2Fn6810:324 Uncaught ReferenceError: subOrganizationPublicationTemporalGraphCommonURL is not defined at ?uri=http%3A%2F%2Flocalhost%3A8080%2Fvivo_i18n%2Findividual%2Fn6810:324:30