-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix(module: drawer): OffsetX and offsetY do not work (#1435) #1448
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1448 +/- ##
=======================================
Coverage 0.00% 0.00%
=======================================
Files 453 453
Lines 28529 28639 +110
=======================================
- Misses 28529 28639 +110
Continue to review full report at Codecov.
|
@@ -1,8 +1,10 @@ | |||
import { getDom } from '../interop'; | |||
|
|||
// @ts-ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May I ask what errors were you getting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error TS2304: Cannot find name 'ResizeObserver'.
Maybe because it's experimental? (In fact, all modern browsers support it except IE)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it is experimental any longer. I cannot find any reference to it being experimental... We do have a detection in place if it does not exist and a fallback to window.resize. On the other hand, I think IE will not be supported by AntBlazor anylonger.
To be honest I never got that error. But I guess there is no harm to have it here though 👍🏻 .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the word "experimental" does not exist in the English document of MDN. Maybe the Chinese document of MDN has not been updated.
@zxyao145 There is a rendering problem with drawer. Please have a look at the gif. Notice that first time To be honest it looks awfully similar to a problem that I was dealing with |
Yes, that's the problem with demo. I'm repairing it locally |
@zxyao145 I think you need to update your branch for the build errors to go away. |
@zxyao145 I found that the closing animation was lost. |
fixed |
The drawer opening animation has a sense of paragraph, not smooth enough. |
* test: DateHelperTests was in the wrong dir (#1529) * fix(module: radio): add defaultChecked and defaultValue (#1494) * fix(module: radio): add defaultChecked and defaultValue * add tests * add test for radio group * feat: Added locale es-ES (#1534) * fix(module: locale): add missing 'filterOptions' node to german locale file (#1562) * docs(module: table): update ‘fork official sample’ demo (#1544) Co-authored-by: James Yeung <[email protected]> * fix(module: npuNumber): include parser in value evaluation (#1567) * fix(module: input): fix & feat & docs & tests: Input general fixes (#1530) * docs(module:input): api correction & comments * fix(module:input): clear icon in sync with antD * fix(module:input): add parameter bordered * fix(module:textarea): add parameter ShowCounter * fix(module:input): Focus with behaviors * docs: add missing docs, rearrange order * fix(module:search): update to match new standard look doc(module:search): add API & demos * fix(module:inputgroup): override style to remove whitespace\ docs: same as react * fix(module:inputpassword): support custom icon docs: same as react * fix(module:input): add blur method docs: add blur method * fix(module:input): add readonly parameter * tests * fix * fix(module:input): sequence correction * Update components/core/Base/AntComponentBase.cs Co-authored-by: James Yeung <[email protected]> * review fixes * fix: clean-up * tests: fix missing mock * fix: focus * fix(module:input): bind clear button to _inputValue * tests: clear button show/hide * tests: package update Co-authored-by: James Yeung <[email protected]> * fix(module: drawer): OffsetX and offsetY do not work (#1435) (#1448) * fix(module: drawer): OffsetX and offsetY do not work (#1435) * docs: demo update * fix: Placement is switched to the relative direction, the animation appears * fix: the bug of closing animation missed * fix: incorrect animation * fix: different behaviors in WASM and Server side Co-authored-by: James Yeung <[email protected]> * docs: Added essential step in get started procedure (#1579) * Added essential step in get started procedure The article is missing the dependency registration. If these dependencies are not registered, the library couldn't be used (at least in my case). * Update getting-started.en-US.md Co-authored-by: James Yeung <[email protected]> * docs: document improvement (#1578) * fix the ordering of menu items * add ueditor link * fix icon document search style * fix affix width for side menu * add faq doc * feat(module: datepicker): Picker clear button event (#1586) * feat(module:datepicker): add OnClearClick eventcallback * docs(module:datepicker): format docs has same behavior as react docs * fix: adjust code to match net 3.1 * fix: eventcallback name fix * Update site/AntDesign.Docs/Demos/Components/DatePicker/doc/index.zh-CN.md Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): align with react (#1588) * fix(module:dropdownbutton): align with react version * doc(module:dropdownbutton): align with react docs * fix(module: table): fix can not close filter by confirm button (#1602) * fix(module: table): fix filter icon wrong highlight, fix filter behavior when there is no input value (#1592) * fix(module:table): fix filter icon wrong highlight * Update Column.razor * fix(module:table): fix filter for no input * fix(module: table): fix filter not applied (#1594) * fix(module: InputNumber): fix error when blur (#1612) * fix(affix): remove wrong event listeners (#1616) * fix(module: table): fix wrong compare operator, remove input components for unary filter operators (#1596) * fix(module:table): fix wrong compare operator * remove whitespace * fix(module:table): remove input component for 'is null' and 'is not null' table filter * Update Column.razor * fix(module:table): allow remove first filter * remove whitespace * Update Column.razor * Update Column.razor Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): dropdown button missing properties & docs fix for Dropdown & Button (#1609) * fix:fix(module:dropdownbutton): add regular button behaviors and properties * fix(module:button): add text type & docs * docs(module:dropdown): version correction * fix(module:overlay): re-add IsButton parameter * fix: add xml comment to public methods * fix(module:overlay): add xml documentation * fix(module:dropdown): chrome block button force * fix(module:button): remove Search parameter Co-authored-by: James Yeung <[email protected]> * fix(module: datepicker): in form behavior (#1617) * i18n: Add missing elements to Dutch locales (#1624) * chore(ci): fix coverage report uploading (#1614) * chore(ci): fix coverage report uploading * continue when fail to upload * docs: update modal and drawer docs (#1625) * docs: update modal docs * docs: modify DropdownInDrawer * docs: update demo for DrawerService and ModalService * fix(module: modal): Modal dialogs not showing anymore (#1635) * Fix issue #1634 - Modal dialogs not showing anymore * Removed unnecessary Search from ButtonProps as part of #1634 fix - Modal dialogs not showing anymore * fix(module: table): fix selection clear (#1632) * fix(module:table): fix selection issues * allow SetSeletion(null) when there is not a Selection column * allow change selection by setting RowDate.Selected * fix(module: checkbox): fix checkbox group in-form behavior (#1637) * fix(module: image): fix the style property position (#1642) * fix(module: tag): Issue solved & align with react antd & tests (#1631) * fix(module:tag): close does not trigger onclick add custom color handling add animation handling add status tags * docs(module:tag): align with react antd * tests(module:tag): add basic tests * fix: style fixes * fix: remove default animation; add custom animation demo * fix: remove ExtraCls and use Class from base * Update components/tag/Tag.razor * clear the open animate after tag opened * translate demo docs Co-authored-by: James Yeung <[email protected]> * changelog 0.8.2 (#1649) * add Visible back to Tag.razor.cs * Update SearchLoading.razor and idnex.en-US.md * Update index.zh-CN.md Co-authored-by: 笨木头 <[email protected]> Co-authored-by: James Yeung <[email protected]> Co-authored-by: Magehernan <[email protected]> Co-authored-by: Andrzej Bakun <[email protected]> Co-authored-by: zxyao <[email protected]> Co-authored-by: black-snake <[email protected]> Co-authored-by: Gregory Loones <[email protected]> Co-authored-by: Razvan Predescu <[email protected]>
* fix(module: drawer): OffsetX and offsetY do not work (#1435) * docs: demo update * fix: Placement is switched to the relative direction, the animation appears * fix: the bug of closing animation missed * fix: incorrect animation * fix: different behaviors in WASM and Server side Co-authored-by: James Yeung <[email protected]>
* test: DateHelperTests was in the wrong dir (#1529) * fix(module: radio): add defaultChecked and defaultValue (#1494) * fix(module: radio): add defaultChecked and defaultValue * add tests * add test for radio group * feat: Added locale es-ES (#1534) * fix(module: locale): add missing 'filterOptions' node to german locale file (#1562) * docs(module: table): update ‘fork official sample’ demo (#1544) Co-authored-by: James Yeung <[email protected]> * fix(module: npuNumber): include parser in value evaluation (#1567) * fix(module: input): fix & feat & docs & tests: Input general fixes (#1530) * docs(module:input): api correction & comments * fix(module:input): clear icon in sync with antD * fix(module:input): add parameter bordered * fix(module:textarea): add parameter ShowCounter * fix(module:input): Focus with behaviors * docs: add missing docs, rearrange order * fix(module:search): update to match new standard look doc(module:search): add API & demos * fix(module:inputgroup): override style to remove whitespace\ docs: same as react * fix(module:inputpassword): support custom icon docs: same as react * fix(module:input): add blur method docs: add blur method * fix(module:input): add readonly parameter * tests * fix * fix(module:input): sequence correction * Update components/core/Base/AntComponentBase.cs Co-authored-by: James Yeung <[email protected]> * review fixes * fix: clean-up * tests: fix missing mock * fix: focus * fix(module:input): bind clear button to _inputValue * tests: clear button show/hide * tests: package update Co-authored-by: James Yeung <[email protected]> * fix(module: drawer): OffsetX and offsetY do not work (#1435) (#1448) * fix(module: drawer): OffsetX and offsetY do not work (#1435) * docs: demo update * fix: Placement is switched to the relative direction, the animation appears * fix: the bug of closing animation missed * fix: incorrect animation * fix: different behaviors in WASM and Server side Co-authored-by: James Yeung <[email protected]> * docs: Added essential step in get started procedure (#1579) * Added essential step in get started procedure The article is missing the dependency registration. If these dependencies are not registered, the library couldn't be used (at least in my case). * Update getting-started.en-US.md Co-authored-by: James Yeung <[email protected]> * docs: document improvement (#1578) * fix the ordering of menu items * add ueditor link * fix icon document search style * fix affix width for side menu * add faq doc * feat(module: datepicker): Picker clear button event (#1586) * feat(module:datepicker): add OnClearClick eventcallback * docs(module:datepicker): format docs has same behavior as react docs * fix: adjust code to match net 3.1 * fix: eventcallback name fix * Update site/AntDesign.Docs/Demos/Components/DatePicker/doc/index.zh-CN.md Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): align with react (#1588) * fix(module:dropdownbutton): align with react version * doc(module:dropdownbutton): align with react docs * fix(module: table): fix can not close filter by confirm button (#1602) * fix(module: table): fix filter icon wrong highlight, fix filter behavior when there is no input value (#1592) * fix(module:table): fix filter icon wrong highlight * Update Column.razor * fix(module:table): fix filter for no input * fix(module: table): fix filter not applied (#1594) * fix(module: InputNumber): fix error when blur (#1612) * fix(affix): remove wrong event listeners (#1616) * fix(module: table): fix wrong compare operator, remove input components for unary filter operators (#1596) * fix(module:table): fix wrong compare operator * remove whitespace * fix(module:table): remove input component for 'is null' and 'is not null' table filter * Update Column.razor * fix(module:table): allow remove first filter * remove whitespace * Update Column.razor * Update Column.razor Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): dropdown button missing properties & docs fix for Dropdown & Button (#1609) * fix:fix(module:dropdownbutton): add regular button behaviors and properties * fix(module:button): add text type & docs * docs(module:dropdown): version correction * fix(module:overlay): re-add IsButton parameter * fix: add xml comment to public methods * fix(module:overlay): add xml documentation * fix(module:dropdown): chrome block button force * fix(module:button): remove Search parameter Co-authored-by: James Yeung <[email protected]> * fix(module: datepicker): in form behavior (#1617) * i18n: Add missing elements to Dutch locales (#1624) * chore(ci): fix coverage report uploading (#1614) * chore(ci): fix coverage report uploading * continue when fail to upload * docs: update modal and drawer docs (#1625) * docs: update modal docs * docs: modify DropdownInDrawer * docs: update demo for DrawerService and ModalService * fix(module: modal): Modal dialogs not showing anymore (#1635) * Fix issue #1634 - Modal dialogs not showing anymore * Removed unnecessary Search from ButtonProps as part of #1634 fix - Modal dialogs not showing anymore * fix(module: table): fix selection clear (#1632) * fix(module:table): fix selection issues * allow SetSeletion(null) when there is not a Selection column * allow change selection by setting RowDate.Selected * fix(module: checkbox): fix checkbox group in-form behavior (#1637) * fix(module: image): fix the style property position (#1642) * fix(module: tag): Issue solved & align with react antd & tests (#1631) * fix(module:tag): close does not trigger onclick add custom color handling add animation handling add status tags * docs(module:tag): align with react antd * tests(module:tag): add basic tests * fix: style fixes * fix: remove default animation; add custom animation demo * fix: remove ExtraCls and use Class from base * Update components/tag/Tag.razor * clear the open animate after tag opened * translate demo docs Co-authored-by: James Yeung <[email protected]> * changelog 0.8.2 (#1649) * add Visible back to Tag.razor.cs * Update SearchLoading.razor and idnex.en-US.md * Update index.zh-CN.md Co-authored-by: 笨木头 <[email protected]> Co-authored-by: James Yeung <[email protected]> Co-authored-by: Magehernan <[email protected]> Co-authored-by: Andrzej Bakun <[email protected]> Co-authored-by: zxyao <[email protected]> Co-authored-by: black-snake <[email protected]> Co-authored-by: Gregory Loones <[email protected]> Co-authored-by: Razvan Predescu <[email protected]>
* fix(module: drawer): OffsetX and offsetY do not work (#1435) * docs: demo update * fix: Placement is switched to the relative direction, the animation appears * fix: the bug of closing animation missed * fix: incorrect animation * fix: different behaviors in WASM and Server side Co-authored-by: James Yeung <[email protected]>
* test: DateHelperTests was in the wrong dir (#1529) * fix(module: radio): add defaultChecked and defaultValue (#1494) * fix(module: radio): add defaultChecked and defaultValue * add tests * add test for radio group * feat: Added locale es-ES (#1534) * fix(module: locale): add missing 'filterOptions' node to german locale file (#1562) * docs(module: table): update ‘fork official sample’ demo (#1544) Co-authored-by: James Yeung <[email protected]> * fix(module: npuNumber): include parser in value evaluation (#1567) * fix(module: input): fix & feat & docs & tests: Input general fixes (#1530) * docs(module:input): api correction & comments * fix(module:input): clear icon in sync with antD * fix(module:input): add parameter bordered * fix(module:textarea): add parameter ShowCounter * fix(module:input): Focus with behaviors * docs: add missing docs, rearrange order * fix(module:search): update to match new standard look doc(module:search): add API & demos * fix(module:inputgroup): override style to remove whitespace\ docs: same as react * fix(module:inputpassword): support custom icon docs: same as react * fix(module:input): add blur method docs: add blur method * fix(module:input): add readonly parameter * tests * fix * fix(module:input): sequence correction * Update components/core/Base/AntComponentBase.cs Co-authored-by: James Yeung <[email protected]> * review fixes * fix: clean-up * tests: fix missing mock * fix: focus * fix(module:input): bind clear button to _inputValue * tests: clear button show/hide * tests: package update Co-authored-by: James Yeung <[email protected]> * fix(module: drawer): OffsetX and offsetY do not work (#1435) (#1448) * fix(module: drawer): OffsetX and offsetY do not work (#1435) * docs: demo update * fix: Placement is switched to the relative direction, the animation appears * fix: the bug of closing animation missed * fix: incorrect animation * fix: different behaviors in WASM and Server side Co-authored-by: James Yeung <[email protected]> * docs: Added essential step in get started procedure (#1579) * Added essential step in get started procedure The article is missing the dependency registration. If these dependencies are not registered, the library couldn't be used (at least in my case). * Update getting-started.en-US.md Co-authored-by: James Yeung <[email protected]> * docs: document improvement (#1578) * fix the ordering of menu items * add ueditor link * fix icon document search style * fix affix width for side menu * add faq doc * feat(module: datepicker): Picker clear button event (#1586) * feat(module:datepicker): add OnClearClick eventcallback * docs(module:datepicker): format docs has same behavior as react docs * fix: adjust code to match net 3.1 * fix: eventcallback name fix * Update site/AntDesign.Docs/Demos/Components/DatePicker/doc/index.zh-CN.md Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): align with react (#1588) * fix(module:dropdownbutton): align with react version * doc(module:dropdownbutton): align with react docs * fix(module: table): fix can not close filter by confirm button (#1602) * fix(module: table): fix filter icon wrong highlight, fix filter behavior when there is no input value (#1592) * fix(module:table): fix filter icon wrong highlight * Update Column.razor * fix(module:table): fix filter for no input * fix(module: table): fix filter not applied (#1594) * fix(module: InputNumber): fix error when blur (#1612) * fix(affix): remove wrong event listeners (#1616) * fix(module: table): fix wrong compare operator, remove input components for unary filter operators (#1596) * fix(module:table): fix wrong compare operator * remove whitespace * fix(module:table): remove input component for 'is null' and 'is not null' table filter * Update Column.razor * fix(module:table): allow remove first filter * remove whitespace * Update Column.razor * Update Column.razor Co-authored-by: James Yeung <[email protected]> * fix(module: dropdown): dropdown button missing properties & docs fix for Dropdown & Button (#1609) * fix:fix(module:dropdownbutton): add regular button behaviors and properties * fix(module:button): add text type & docs * docs(module:dropdown): version correction * fix(module:overlay): re-add IsButton parameter * fix: add xml comment to public methods * fix(module:overlay): add xml documentation * fix(module:dropdown): chrome block button force * fix(module:button): remove Search parameter Co-authored-by: James Yeung <[email protected]> * fix(module: datepicker): in form behavior (#1617) * i18n: Add missing elements to Dutch locales (#1624) * chore(ci): fix coverage report uploading (#1614) * chore(ci): fix coverage report uploading * continue when fail to upload * docs: update modal and drawer docs (#1625) * docs: update modal docs * docs: modify DropdownInDrawer * docs: update demo for DrawerService and ModalService * fix(module: modal): Modal dialogs not showing anymore (#1635) * Fix issue #1634 - Modal dialogs not showing anymore * Removed unnecessary Search from ButtonProps as part of #1634 fix - Modal dialogs not showing anymore * fix(module: table): fix selection clear (#1632) * fix(module:table): fix selection issues * allow SetSeletion(null) when there is not a Selection column * allow change selection by setting RowDate.Selected * fix(module: checkbox): fix checkbox group in-form behavior (#1637) * fix(module: image): fix the style property position (#1642) * fix(module: tag): Issue solved & align with react antd & tests (#1631) * fix(module:tag): close does not trigger onclick add custom color handling add animation handling add status tags * docs(module:tag): align with react antd * tests(module:tag): add basic tests * fix: style fixes * fix: remove default animation; add custom animation demo * fix: remove ExtraCls and use Class from base * Update components/tag/Tag.razor * clear the open animate after tag opened * translate demo docs Co-authored-by: James Yeung <[email protected]> * changelog 0.8.2 (#1649) * add Visible back to Tag.razor.cs * Update SearchLoading.razor and idnex.en-US.md * Update index.zh-CN.md Co-authored-by: 笨木头 <[email protected]> Co-authored-by: James Yeung <[email protected]> Co-authored-by: Magehernan <[email protected]> Co-authored-by: Andrzej Bakun <[email protected]> Co-authored-by: zxyao <[email protected]> Co-authored-by: black-snake <[email protected]> Co-authored-by: Gregory Loones <[email protected]> Co-authored-by: Razvan Predescu <[email protected]>
🤔 This is a ...
🔗 Related issue link
#1435
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge