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

Makefile: Update Cockpit lib to 3b1a41701581de0ed6826191b6d72e83 #1894

Merged
merged 5 commits into from
Nov 8, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 4, 2024

Jelle van der Waa (2):
test: restore fstab/crypttab for RHEL 8 storage tests
lib: restrict SASS plugin to scss files

Marius Vollmer (9):
lib: Copy TypeaheadSelect into our code
lib: Fix TypeScript error in TypeaheadSelect
lib: Optionally trust selected property in TypeaheadSelect
lib: Support dividers with TypeaheadSelect
lib: Hide clear button in TypeaheadSelect when it would do nothing
lib: Simplify and translate UI texts in TypeaheadSelect
storage: Use TypeaheadSelect instead of deprecated Select
lib: Use TypeaheadSelect instead of deprecated Select for timezones
lib: Use TypeaheadSelect instead of deprecated Select for FileAutoComplete

Martin Pitt (3):
lib: Properly type getLastlog2
lib: Add typing for cockpit.jump() and cockpit.dbus().call()
lib: Initial typing of python.js

@github-actions github-actions bot added the bot label Nov 4, 2024
@github-actions github-actions bot changed the title [no-test] Makefile: Update Cockpit lib to 3b1a41701581de0ed6826191b6d72e83 Makefile: Update Cockpit lib to 3b1a41701581de0ed6826191b6d72e83 Nov 4, 2024
@allisonkarlitskaya allisonkarlitskaya force-pushed the cockpit-lib-update-cockpit-lib-20241104-183546 branch from b7679fa to 4a1ca3f Compare November 4, 2024 18:35
@martinpitt
Copy link
Member

This needs adjustment to the newly ported TypeaheadSelect.

@jelly
Copy link
Member

jelly commented Nov 5, 2024

This needs adjustment to the newly ported TypeaheadSelect.

Yup, already started locally. 1/X tests fixed :)

cockpituous and others added 2 commits November 6, 2024 16:58
This does not make a lot of sense as it will show no such file or
directory. Only set it when we have a valid location to set.
@jelly jelly force-pushed the cockpit-lib-update-cockpit-lib-20241104-183546 branch from 4a1ca3f to 864a16d Compare November 6, 2024 17:22
@jelly
Copy link
Member

jelly commented Nov 6, 2024

Firefox specific issues left

The FileAutoComplete component now uses a non-deprecated Select which
has a different internal DOM state.

The create text was also adjusted to no longer quote the path.
Instead of asserting it exists, validate that the correct text is shown.
@jelly jelly force-pushed the cockpit-lib-update-cockpit-lib-20241104-183546 branch from 7a9e642 to 98e6471 Compare November 7, 2024 19:02
@jelly jelly requested a review from martinpitt November 8, 2024 08:01
@jelly
Copy link
Member

jelly commented Nov 8, 2024

The centos-10 coredump here is the same as what blocks are gating and needs to be investigated

Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

This pixel change is a bit sad -- we now have two different "X" buttons in inputs in the same dialog. Can we port the "operating system" one as well?

@jelly
Copy link
Member

jelly commented Nov 8, 2024

Thanks!

This pixel change is a bit sad -- we now have two different "X" buttons in inputs in the same dialog. Can we port the "operating system" one as well?

That is what Marius intended to work on but it needs further enhancements of the Typeahead to support grouping? This one:
#1872

@jelly jelly merged commit de46d0f into main Nov 8, 2024
28 of 29 checks passed
@jelly jelly deleted the cockpit-lib-update-cockpit-lib-20241104-183546 branch November 8, 2024 08:51
@mvollmer
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants