Skip to content
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

[ENH] Datasets: Add language selection #6358

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Mar 6, 2023

Issue

Data sets in the repository now have a language flag, and Data Sets widget allows user to filter files by language.

Default is English, but translations of Orange to other languages can change this default.

Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Merging #6358 (46803f8) into master (dfaca7e) will increase coverage by 0.00%.
The diff coverage is 94.11%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6358   +/-   ##
=======================================
  Coverage   87.39%   87.40%           
=======================================
  Files         317      317           
  Lines       68238    68289   +51     
=======================================
+ Hits        59640    59688   +48     
- Misses       8598     8601    +3     

@janezd janezd force-pushed the datasets-language branch from b77bd36 to 4d32ec6 Compare March 6, 2023 19:10
@BlazZupan
Copy link
Contributor

Please also change the icon that shows that the data set is local but not loaded. Currently, there is a filled grey circle to denote this, but the difference with the filled green circle is not substantial [for instance, to the people with a degree of color blindness]. It will help if the grey circles are empty.

@janezd
Copy link
Contributor Author

janezd commented Mar 10, 2023

Please also change the icon that shows that the data set is local but not loaded.

This will be another PR because it requires changing (and then releasing) orange-widget-base.

@janezd janezd marked this pull request as ready for review March 10, 2023 11:05
@janezd janezd force-pushed the datasets-language branch from 4d32ec6 to 46803f8 Compare March 10, 2023 13:21
@lanzagar lanzagar changed the title Datasets: Add language selection [ENH] Datasets: Add language selection Mar 10, 2023
@lanzagar lanzagar merged commit 0348082 into biolab:master Mar 10, 2023
@janezd janezd mentioned this pull request Mar 11, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants