Tuesday, March 25, 2025
3 changes
Enhancements to existing features
This update removes unused animation code from the VoIP menu area. It helps keep the interface code cleaner and avoids relying on an animation name that browsers do not recognize, with no expected change for users.
Original PR description
This commit removes the unused `o-voip-SystrayItem-vibrateAnimation` CSS animation and also removes the `vibrate` animation as it is an unknown CSS animation-name. Reference: - https://developer.mozilla.org/en-US/docs/Web/CSS/animation
This update streamlines automated testing across Point of Sale and related country-specific modules by removing duplicate checks, merging overlapping test scenarios, and splitting large tests into easier-to-maintain parts. It helps improve test reliability and maintainability without changing business functionality.
Original PR description
Cleaning all tours and tests in PoS related modules, removing duplicated tests and tours, and fixing some issues in the tests. Deleted / merged tests: - The test `test_01_pos_basic_order` is now splitted in separate method to easier debug and maintain. - The test `test_06_pos_discount_display_with_multiple_pricelist` is merged with `test_01_pos_basic_order` in `pos_pricelist` tour. - The test `test_product_combo_price` is removed since test content is duplicate in `test_07_product_combo` which also tests prices. - The test `test_autofill_cash_count` is merged with `test_properly_display_price` since it is a sub-test of it. - The test `test_product_search_2` is merged with `test_product_search` - Tests `test_rounding_up`, `test_rounding_down` and `test_rounding_half_up` are deleted since class `TestPosCashRounding` already tests rounding. taskId: 4467373
The barcode interface has been tidied up by removing an animation that was incorrectly defined and therefore not working. This reduces unnecessary styling code without changing how users interact with barcode operations.
Original PR description
This commit removes the malformed (and thus unapplied) `highlighting-flash-primary` CSS animation.
In a nutshell, the animation as written in SCSS doesn't interpret the `theme-color("primary")` call and write it as-is in the generated CSS... which, in return, is an unknown CSS value.
Note: this commit also removes the `highlighting-flash-white` animation as it isn't used.