-
Notifications
You must be signed in to change notification settings - Fork 93
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
Autocomplete problems after Bootstrap 5 upgrade #1301
Comments
Ah, that's an easy one to fix. @osma should we make the default to be that black/dark-gray from finto.fi, or leave as is? Here's where we can change it to any other color: Skosmos/resource/css/styles.css Line 828 in df06b59
|
Right. Looking closer, it seems that this CSS block was added in PR #1182: Skosmos/resource/css/styles.css Lines 826 to 831 in df06b59
I'm not sure I understand the reason for this block. To me it seems that the only effective change is the text color, which is the problematic one. I tested dropping the whole block and couldn't see any problems with that. So I suggest that the block should be deleted, unless there is a reason for some of those rules. |
PR #1303 fixed the functional issues (2, 3). Reopening because of the font color issue (1). Also the font size on the buttons ("English", "Search" in above screenshots) is slightly larger (16px / 1rem) than it used to be (14px), which I hadn't noticed before. I'll soon open a PR fixing these as well. |
At which URL did you encounter the problem?
https://dev.finto.fi/yso/en/
What steps will reproduce the problem?
What is the expected output? What do you see instead?
This is how it works in finto.fi, running Skosmos 2.14:
Here is how it looks with current master branch (dev.finto.fi):
There are at least three problems:
This most likely broke with the Bootstrap 5 upgrade (PR #1182), which included an upgrade of the Typeahead.js widget.
Ping @kinow
What browser did you use? (eg. Firefox, Chrome, Safari, Internet explorer)
Firefox
The text was updated successfully, but these errors were encountered: