projects

Clone Tools
  • last updated 11 mins ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
FLEX-35267 Updated unit test - made it easier to read, and now we're also testing GridColumnHeaderGroup.containsGlobalCoordinates(). For the test to pass I've improved the logic in GridHeaderViewLayout.getHeaderIndexAt() which, in turn, made a few functions redundant in GridColumnHeaderGroup. All tests currently pass.

FLEX-35260 Now we are taking into account paddingTop and paddingBottom in getHeaderIndexAt(). Unit test updated as well (and passes, including a second data grid configuration).

FLEX-35260 Now the unit test passes, having been updated to check the column header view boundaries. Other changes: -Now we're ignoring the grid header's right padding, because it doesn't make a visual difference (except in the edge case where, together with the left and right padding, the column widths are wider than the grid's width, in which case the grid header expands beyond the grid limits), but we are starting to take into account bottom and top padding. -Renamed some variables and functions for clarity in GridColumnHeaderView and GridDimensions. -Improved error messages in the unit test

Merge branch 'release4.16.0'

FLEX-35260 Now in the unit test we're also verifying areCoordinatesOverAHeaderView(). Currently fails (for point "b").

FLEX-35260 GridHeaderViewLayout.getHeaderIndexAt() (which calls GridDimensions.getColumnIndexAt()) used to return a valid headerIndex (when it was expected to return -1) for an y < 0. Fixed.

FLEX-35260 GridHeaderViewLayout.getHeaderIndexAt() (which calls GridDimensions.getColumnIndexAt()) used to return a valid headerIndex (when it was expected to return -1) for an x < 0. Fixed.

FLEX-35267 FLEX-35260 FLEX-35029 1. GridHeaderViewLayout.getHeaderIndexAt() now verifies if the coordinates are over the GridColumnHeaderView's left or right padding. 2. Renamed GridColumnHeaderGroup.mouseEventHeaderView() to getHeaderViewUnderGlobalCoordinates(), while also changing the parameter to be a Point instead of a MouseEvent. This is so that areCoordinatesOverAHeaderView(), areCoordinatesOverLeftPadding() and areCoordinatesOverRightPadding() can use it too. 3. Expanded some variable names in GridColumnHeaderGroup.

FLEX-35267 FLEX-35260 Added a unit test which can verify the column-index and header-index related functions more thoroughly. Currently it fails, as expected.

FLEX-27509 CAUSE: In an AdvancedDataGrid with non-text item renderers AdvancedDataGrid.expandItem() applies masks onto those item renderers when it needs to close a node. At the same time, AdvancedListBase has its own custom mechanism for using masks to scroll vertically in a more efficient way. In order to detect whether it's used this mechanism, and reset the changes (detection happens in AdvancedListBase.removeClipMask(), while the mechanism is applied in addClipMask()), it simply asks whether the non-text item renderer has a mask applied. However, this mask could have been applied by the expandItem() mechanism mentioned above, which means that addClipMask() will never have been called. As a result, itemMaskFreeList is still null, which leads to the fatal.

SOLUTION: Ideally there should be a flag that specifies whether AdvancedListBase.addClipMask() has applied the masking mechanism. But for now we can simply check that itemMaskFreeList isn't null before we use it.

FLEX-27509 CAUSE: In an AdvancedDataGrid with non-text item renderers AdvancedDataGrid.expandItem() applies masks onto those item renderers when it needs to close a node. At the same time, AdvancedListBase has its own custom mechanism for using masks to scroll vertically in a more efficient way. In order to detect whether it's used this mechanism, and reset the changes (detection happens in AdvancedListBase.removeClipMask(), while the mechanism is applied in addClipMask()), it simply asks whether the non-text item renderer has a mask applied. However, this mask could have been applied by the expandItem() mechanism mentioned above, which means that addClipMask() will never have been called. As a result, itemMaskFreeList is still null, which leads to the fatal.

SOLUTION: Ideally there should be a flag that specifies whether AdvancedListBase.addClipMask() has applied the masking mechanism. But for now we can simply check that itemMaskFreeList isn't null before we use it.

Fix regression issue. Turns out LabelUtil.itemToLabel has side effects and text was not updated on control unless called

    • -5
    • +7
    ./spark/src/spark/components/DropDownList.as
FLEX-34880 Added an asdoc recommendation that developers use Sort and SortField as immutable objects, although the current API allows for mutability.

    • -0
    • +5
    ./framework/src/mx/collections/Sort.as
    • -0
    • +5
    ./framework/src/mx/collections/SortField.as
    • -0
    • +5
    ./spark/src/spark/collections/SortField.as
Revert FLEX-34880 part 3

This reverts commit 2b09e327281211d26d65dd5d061b02d645cbdd39.

    • -1
    • +7
    ./framework/src/mx/collections/ISortField.as
    • -2
    • +2
    ./framework/src/mx/collections/SortField.as
    • -5
    • +4
    ./spark/src/spark/collections/SortField.as
  1. … 1 more file in changeset.
Revert FLEX-34880 part 2

This reverts commit ff4067f56651082cecbe062f9cfd2aedbe44bf60.

    • -2
    • +2
    ./framework/tests/ListCollectionView_FLEX_34837_Tests.as
Revert FLEX-34880 part 1 This reverts commit 0b5a634dabb00c04a492a08375e29c6885c1486f.

    • -0
    • +23
    ./framework/src/mx/collections/ISortField.as
    • -6
    • +6
    ./framework/src/mx/collections/SortField.as
    • -2
    • +1
    ./spark/src/spark/components/DataGrid.as
  1. … 3 more files in changeset.
FLEX-34811 allow apostrophe in user name part of email address

FLEX-34738 Fix BusyIndicator so the skin / timer is removed when it's removed from stage while still spinning.

FLEX-35122: Update to FontAwesome 4.7

    • -16
    • +106
    ./flatspark/src/flatspark/utils/AwesomeUtils.as
FLEX-34880 Placed the other ISortField setters behind mx_internal and removed them from the interface.

    • -23
    • +0
    ./framework/src/mx/collections/ISortField.as
    • -6
    • +6
    ./framework/src/mx/collections/SortField.as
    • -1
    • +2
    ./spark/src/spark/components/DataGrid.as
  1. … 3 more files in changeset.
FLEX-34880 Omitted to change a unit test.

    • -2
    • +2
    ./framework/tests/ListCollectionView_FLEX_34837_Tests.as
FLEX-34880 Placed only one state setter (SortField.compareFunction) behind mx_internal (thus also renaming it to SortField.compareFunction_) to provide the template for the others. Also removed it from the interface (ISortField).

    • -7
    • +1
    ./framework/src/mx/collections/ISortField.as
    • -2
    • +2
    ./framework/src/mx/collections/SortField.as
    • -4
    • +5
    ./spark/src/spark/collections/SortField.as
  1. … 1 more file in changeset.
fix as docs issue with mismatched tags

    • -1
    • +1
    ./framework/src/mx/utils/ObjectUtil.as
FLEX-26808 Increased a test function's timeout and added a bit more information to see why another is failing.

FLEX-34088 Using addChild instead of addElement for FlexUnit to get the test to run.

FLEX-35126 Added a unit test to reprouce the bug. Currently it passes, as expected. (But if I manually put back the fix for FLEX-34088, it fails, as it should, meaning it actually tests the correct bug.)

    • -0
    • +182
    ./spark/tests/spark/components/DropDownList_FLEX_35126_Tests.as
FLEX-35126 Removing unused variables, making elligible functions static, correcting an asdoc typo, and removing superfluous multiplication by 1.

Fix typo

FLEX-34088 The fix for this bug became superfluous when the fix for FLEX-34078 was implemented, which also catered to this bug. So it can be safely removed (and should, because it causes other bugs - see the ticket).

FLEX-34088 Added unit test to reproduce the bug. Currently it passes, but I tested locally with some modifications to DropDownController and can certify it does indeed fail when it should - namely, when both the fix for FLEX-34088 and FLEX-34078 are removed. In other words, the fix for FLEX-34078 also fixes this issue, making the current fix superfluous. I'll remove it in the next commit.