You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/01/13 17:04:49 UTC
[GitHub] [superset] kkucharc opened a new pull request #12498: fix: row component handler is visible
kkucharc opened a new pull request #12498:
URL: https://github.com/apache/superset/pull/12498
### SUMMARY
In dashboard's edit mode menu with `delete` and `move` on left was cut by left end of page/browser.
I made equal margin on both sides. The menu is closer the edited row.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
**Before:**
Left margin was more narrow than right one.
<img width="1341" alt="Screenshot 2021-01-13 at 17 47 17" src="https://user-images.githubusercontent.com/2536609/104482841-b6e13f00-55c7-11eb-83ed-2f1707cc5197.png">
Menu with `move` and `delete` in dashboard edit was truncated:
<img width="969" alt="Screenshot 2021-01-13 at 17 47 37" src="https://user-images.githubusercontent.com/2536609/104482812-ad57d700-55c7-11eb-92eb-2f8656ec05f0.png">
**After:**
_Without native filters:_
Preview mode:
![Screenshot 2021-01-13 at 17 39 47](https://user-images.githubusercontent.com/2536609/104483484-733b0500-55c8-11eb-977a-d539d1f3bd79.png)
Edit mode:
![Screenshot 2021-01-13 at 17 39 11](https://user-images.githubusercontent.com/2536609/104483450-6c13f700-55c8-11eb-9e41-49aafd7213c1.png)
_With native filters:_
Preview mode:
![Screenshot 2021-01-13 at 17 41 36](https://user-images.githubusercontent.com/2536609/104483316-45ee5700-55c8-11eb-8b7e-29d12365a643.png)
Edit mode:
![Screenshot 2021-01-13 at 17 42 47](https://user-images.githubusercontent.com/2536609/104483325-4850b100-55c8-11eb-95cf-875b8d65576e.png)
### TEST PLAN
1. Go to any dashboard and check margins
1a. Go to edit mode of dashboard and try edit row of dashboard (on left there will be icon to delete or to move chart)
To test native filters:
Go to `supserset/config.py` and change `"DASHBOARD_NATIVE_FILTERS": False,` -> `"DASHBOARD_NATIVE_FILTERS": True,`
### ADDITIONAL INFORMATION
- [x] Has associated issue: close: #12481
- [ ] Changes UI
- [ ] Requires DB Migration.
- [ ] Confirm DB Migration upgrade and downgrade tested.
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
cc: @agatapst Can you take a look? I made some changes in your code.
cc: @junlincc @adam-stasiak
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] agatapst edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
agatapst edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760057175
@kkucharc regarding native filters:
- when Filter Bar is hidden, the sidebar should look the same in Dashboard page and Explore page and it does 👍
Explore sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 46](https://user-images.githubusercontent.com/47450693/104567842-9066e680-564f-11eb-8505-90c2a23d8b2e.png)
Dashboard sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 35](https://user-images.githubusercontent.com/47450693/104567872-9957b800-564f-11eb-93a1-642bacbb03d2.png)
- when Filter Bar is open:
in your PR:
![Zrzut ekranu 2021-01-14 o 10 05 37](https://user-images.githubusercontent.com/47450693/104568321-3f0b2700-5650-11eb-8139-9ae6dcfc1f4b.png)
master:
![Zrzut ekranu 2021-01-14 o 10 00 04](https://user-images.githubusercontent.com/47450693/104568312-3a467300-5650-11eb-8677-69738669ac40.png)
I can see there is bigger distance between Filter Bar and Dashboard in your PR, but in my opinion both versions look just fine. 🙂 If the cost of smaller distance between Native Filter Bar and Dashboard are UI issues in some other component, then I am fine with it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] junlincc commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
junlincc commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760329165
<img width="498" alt="Screen Shot 2021-01-14 at 8 58 38 AM" src="https://user-images.githubusercontent.com/67837651/104622996-c358ac80-5646-11eb-81da-bb38a7b0875f.png">
can we have the icons perfectly centered in the page by adding more padding on the right side?
<img width="60" alt="Screen Shot 2021-01-14 at 8 59 54 AM" src="https://user-images.githubusercontent.com/67837651/104623137-e84d1f80-5646-11eb-8f86-25a7e982b8c2.png">
<img width="662" alt="Screen Shot 2021-01-14 at 9 01 58 AM" src="https://user-images.githubusercontent.com/67837651/104623415-32360580-5647-11eb-8c8f-c47b8b7a92f8.png">
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] agatapst commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
agatapst commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760057175
@kkucharc regarding native filters:
- when Filter Bar is hidden, the sidebar should look the same in Dashboard page and Explore page and it does 👍
Explore sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 46](https://user-images.githubusercontent.com/47450693/104567842-9066e680-564f-11eb-8505-90c2a23d8b2e.png)
Dashboard sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 35](https://user-images.githubusercontent.com/47450693/104567872-9957b800-564f-11eb-93a1-642bacbb03d2.png)
- when Filter Bar is open:
in your PR:
![Zrzut ekranu 2021-01-14 o 10 05 37](https://user-images.githubusercontent.com/47450693/104568321-3f0b2700-5650-11eb-8139-9ae6dcfc1f4b.png)
master:
![Zrzut ekranu 2021-01-14 o 10 00 04](https://user-images.githubusercontent.com/47450693/104568312-3a467300-5650-11eb-8677-69738669ac40.png)
I can see there is bigger distance between Filter Bar and Dashboard in your PR, but in my opinion both versions look just fine. If the cost of smaller distance between Native Filter Bar and Dashboard is UI issues in some other component, then I am totally fine with it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] kkucharc commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
kkucharc commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-761060249
Thanks @rusackas @junlincc for the test and review. I centered horizontally those icons, rebased and removed IMO redundant (top-bottom) padding. Now it looks like this:
![Screenshot 2021-01-15 at 17 47 14](https://user-images.githubusercontent.com/2536609/104755765-2b001c00-575b-11eb-854e-006b5270c838.png)
![Screenshot 2021-01-15 at 17 47 34](https://user-images.githubusercontent.com/2536609/104755775-2d627600-575b-11eb-8d7e-f09be086768c.png)
![Screenshot 2021-01-15 at 17 52 12](https://user-images.githubusercontent.com/2536609/104755779-2f2c3980-575b-11eb-83c8-bccd3823969f.png)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (5fdbd57) into [master](https://codecov.io/gh/apache/superset/commit/2d10a2c089d0d49c0efd4378b99864108c9afee2?el=desc) (2d10a2c) will **decrease** coverage by `0.28%`.
> The diff coverage is `100.00%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 62.66% 62.37% -0.29%
==========================================
Files 1014 1015 +1
Lines 49593 49591 -2
Branches 4839 4839
==========================================
- Hits 31075 30931 -144
- Misses 18320 18462 +142
Partials 198 198
```
| Flag | Coverage Δ | |
|---|---|---|
| javascript | `60.73% <100.00%> (+<0.01%)` | :arrow_up: |
| python | `63.44% <ø> (-0.48%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...tend/src/dashboard/components/DashboardBuilder.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL0Rhc2hib2FyZEJ1aWxkZXIuanN4) | `76.54% <100.00%> (ø)` | |
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/presto.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3ByZXN0by5weQ==) | `70.56% <0.00%> (-11.48%)` | :arrow_down: |
| [superset/views/database/mixins.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvZGF0YWJhc2UvbWl4aW5zLnB5) | `80.70% <0.00%> (-1.76%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `88.04% <0.00%> (-0.82%)` | :arrow_down: |
| [superset/reports/notifications/base.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvcmVwb3J0cy9ub3RpZmljYXRpb25zL2Jhc2UucHk=) | `95.00% <0.00%> (-0.46%)` | :arrow_down: |
| [superset/migrations/shared/security\_converge.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbWlncmF0aW9ucy9zaGFyZWQvc2VjdXJpdHlfY29udmVyZ2UucHk=) | `86.82% <0.00%> (-0.11%)` | :arrow_down: |
| [superset/connectors/sqla/models.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9zcWxhL21vZGVscy5weQ==) | `90.57% <0.00%> (-0.02%)` | :arrow_down: |
| [superset/sql\_parse.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvc3FsX3BhcnNlLnB5) | `99.36% <0.00%> (-0.01%)` | :arrow_down: |
| ... and [8 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [d70db43...5fdbd57](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (e2a2925) into [master](https://codecov.io/gh/apache/superset/commit/b22e45810866f874bf36b082bdb866cfe9614b40?el=desc) (b22e458) will **decrease** coverage by `7.81%`.
> The diff coverage is `100.00%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 66.85% 59.04% -7.82%
==========================================
Files 1015 959 -56
Lines 49575 46812 -2763
Branches 5079 4344 -735
==========================================
- Hits 33143 27639 -5504
- Misses 16302 19173 +2871
+ Partials 130 0 -130
```
| Flag | Coverage Δ | |
|---|---|---|
| cypress | `50.56% <100.00%> (-0.43%)` | :arrow_down: |
| javascript | `?` | |
| python | `63.81% <ø> (-0.33%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...tend/src/dashboard/components/DashboardBuilder.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL0Rhc2hib2FyZEJ1aWxkZXIuanN4) | `86.41% <100.00%> (ø)` | |
| [...uperset-frontend/src/dashboard/util/dnd-reorder.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2RuZC1yZW9yZGVyLmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...rset-frontend/src/dashboard/util/getEmptyLayout.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2dldEVtcHR5TGF5b3V0Lmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...dashboard/components/resizable/ResizableHandle.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL3Jlc2l6YWJsZS9SZXNpemFibGVIYW5kbGUuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [.../src/dashboard/util/getFilterScopeFromNodesTree.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2dldEZpbHRlclNjb3BlRnJvbU5vZGVzVHJlZS5qcw==) | `0.00% <0.00%> (-93.48%)` | :arrow_down: |
| [...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3ZpZXdzL0NSVUQvYWxlcnQvRXhlY3V0aW9uTG9nLnRzeA==) | `11.76% <0.00%> (-88.24%)` | :arrow_down: |
| [...src/dashboard/components/gridComponents/Header.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL2dyaWRDb21wb25lbnRzL0hlYWRlci5qc3g=) | `10.52% <0.00%> (-86.85%)` | :arrow_down: |
| [superset-frontend/src/components/IconTooltip.tsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2NvbXBvbmVudHMvSWNvblRvb2x0aXAudHN4) | `13.33% <0.00%> (-86.67%)` | :arrow_down: |
| [...rc/dashboard/components/gridComponents/Divider.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL2dyaWRDb21wb25lbnRzL0RpdmlkZXIuanN4) | `13.33% <0.00%> (-86.67%)` | :arrow_down: |
| [...end/src/SqlLab/components/ExploreResultsButton.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9jb21wb25lbnRzL0V4cGxvcmVSZXN1bHRzQnV0dG9uLmpzeA==) | `8.00% <0.00%> (-84.00%)` | :arrow_down: |
| ... and [401 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [b22e458...e2a2925](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (5fdbd57) into [master](https://codecov.io/gh/apache/superset/commit/2d10a2c089d0d49c0efd4378b99864108c9afee2?el=desc) (2d10a2c) will **decrease** coverage by `0.29%`.
> The diff coverage is `100.00%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 62.66% 62.36% -0.30%
==========================================
Files 1014 1015 +1
Lines 49593 49606 +13
Branches 4839 4839
==========================================
- Hits 31075 30938 -137
- Misses 18320 18470 +150
Partials 198 198
```
| Flag | Coverage Δ | |
|---|---|---|
| javascript | `60.73% <100.00%> (+<0.01%)` | :arrow_up: |
| python | `63.43% <ø> (-0.49%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...tend/src/dashboard/components/DashboardBuilder.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL0Rhc2hib2FyZEJ1aWxkZXIuanN4) | `76.54% <100.00%> (ø)` | |
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/presto.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3ByZXN0by5weQ==) | `70.56% <0.00%> (-11.48%)` | :arrow_down: |
| [superset/views/database/mixins.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvZGF0YWJhc2UvbWl4aW5zLnB5) | `80.70% <0.00%> (-1.76%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `88.04% <0.00%> (-0.82%)` | :arrow_down: |
| [superset/db\_engine\_specs/solr.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3NvbHIucHk=) | `100.00% <0.00%> (ø)` | |
| [...rset-frontend/src/explore/components/SaveModal.tsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvY29tcG9uZW50cy9TYXZlTW9kYWwudHN4) | `86.04% <0.00%> (+0.16%)` | :arrow_up: |
| [superset/db\_engine\_specs/pinot.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3Bpbm90LnB5) | `70.73% <0.00%> (+2.31%)` | :arrow_up: |
| [superset/db\_engine\_specs/postgres.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3Bvc3RncmVzLnB5) | `96.05% <0.00%> (+9.21%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [d70db43...5fdbd57](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] junlincc edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
junlincc edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760329165
can we have the icons perfectly centered in the page by adding more padding on the right side? @kkucharc
<img width="60" alt="Screen Shot 2021-01-14 at 8 59 54 AM" src="https://user-images.githubusercontent.com/67837651/104623137-e84d1f80-5646-11eb-8f86-25a7e982b8c2.png">
<img width="662" alt="Screen Shot 2021-01-14 at 9 01 58 AM" src="https://user-images.githubusercontent.com/67837651/104623415-32360580-5647-11eb-8c8f-c47b8b7a92f8.png">
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] agatapst edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
agatapst edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760057175
@kkucharc regarding native filters:
- when Filter Bar is hidden, the sidebar should look the same in Dashboard page and Explore page and it does 👍
Explore sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 46](https://user-images.githubusercontent.com/47450693/104567842-9066e680-564f-11eb-8505-90c2a23d8b2e.png)
Dashboard sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 35](https://user-images.githubusercontent.com/47450693/104567872-9957b800-564f-11eb-93a1-642bacbb03d2.png)
- when Filter Bar is open:
in your PR:
![Zrzut ekranu 2021-01-14 o 10 05 37](https://user-images.githubusercontent.com/47450693/104568321-3f0b2700-5650-11eb-8139-9ae6dcfc1f4b.png)
master:
![Zrzut ekranu 2021-01-14 o 10 00 04](https://user-images.githubusercontent.com/47450693/104568312-3a467300-5650-11eb-8677-69738669ac40.png)
I can see there is bigger distance between Filter Bar and Dashboard in your PR, but in my opinion both versions look just fine. 🙂 If the cost of smaller distance between Native Filter Bar and Dashboard are UI issues in some other component, then I am totally fine with it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (5fdbd57) into [master](https://codecov.io/gh/apache/superset/commit/2d10a2c089d0d49c0efd4378b99864108c9afee2?el=desc) (2d10a2c) will **increase** coverage by `0.78%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
+ Coverage 62.66% 63.44% +0.78%
==========================================
Files 1014 486 -528
Lines 49593 29943 -19650
Branches 4839 0 -4839
==========================================
- Hits 31075 18997 -12078
+ Misses 18320 10946 -7374
+ Partials 198 0 -198
```
| Flag | Coverage Δ | |
|---|---|---|
| javascript | `?` | |
| python | `63.44% <ø> (-0.48%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/presto.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3ByZXN0by5weQ==) | `70.56% <0.00%> (-11.48%)` | :arrow_down: |
| [superset/views/database/mixins.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvZGF0YWJhc2UvbWl4aW5zLnB5) | `80.70% <0.00%> (-1.76%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `88.04% <0.00%> (-0.82%)` | :arrow_down: |
| [superset/reports/notifications/base.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvcmVwb3J0cy9ub3RpZmljYXRpb25zL2Jhc2UucHk=) | `95.00% <0.00%> (-0.46%)` | :arrow_down: |
| [superset/migrations/shared/security\_converge.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbWlncmF0aW9ucy9zaGFyZWQvc2VjdXJpdHlfY29udmVyZ2UucHk=) | `86.82% <0.00%> (-0.11%)` | :arrow_down: |
| [superset/connectors/sqla/models.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9zcWxhL21vZGVscy5weQ==) | `90.57% <0.00%> (-0.02%)` | :arrow_down: |
| [superset/sql\_parse.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvc3FsX3BhcnNlLnB5) | `99.36% <0.00%> (-0.01%)` | :arrow_down: |
| [superset/errors.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZXJyb3JzLnB5) | `100.00% <0.00%> (ø)` | |
| ... and [534 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [d70db43...5fdbd57](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] ktmud commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
ktmud commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759715815
How does it look like when the native filter is expanded?
I filed a related bug report yesterday: https://github.com/apache/superset/issues/12486
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (e2a2925) into [master](https://codecov.io/gh/apache/superset/commit/b22e45810866f874bf36b082bdb866cfe9614b40?el=desc) (b22e458) will **decrease** coverage by `7.67%`.
> The diff coverage is `100.00%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 66.85% 59.17% -7.68%
==========================================
Files 1015 959 -56
Lines 49575 46812 -2763
Branches 5079 4344 -735
==========================================
- Hits 33143 27703 -5440
- Misses 16302 19109 +2807
+ Partials 130 0 -130
```
| Flag | Coverage Δ | |
|---|---|---|
| cypress | `50.94% <100.00%> (-0.05%)` | :arrow_down: |
| javascript | `?` | |
| python | `63.81% <ø> (-0.33%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...tend/src/dashboard/components/DashboardBuilder.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL0Rhc2hib2FyZEJ1aWxkZXIuanN4) | `86.41% <100.00%> (ø)` | |
| [...uperset-frontend/src/dashboard/util/dnd-reorder.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2RuZC1yZW9yZGVyLmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...rset-frontend/src/dashboard/util/getEmptyLayout.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2dldEVtcHR5TGF5b3V0Lmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...dashboard/components/resizable/ResizableHandle.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL3Jlc2l6YWJsZS9SZXNpemFibGVIYW5kbGUuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [.../src/dashboard/util/getFilterScopeFromNodesTree.js](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC91dGlsL2dldEZpbHRlclNjb3BlRnJvbU5vZGVzVHJlZS5qcw==) | `0.00% <0.00%> (-93.48%)` | :arrow_down: |
| [...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3ZpZXdzL0NSVUQvYWxlcnQvRXhlY3V0aW9uTG9nLnRzeA==) | `11.76% <0.00%> (-88.24%)` | :arrow_down: |
| [...src/dashboard/components/gridComponents/Header.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL2dyaWRDb21wb25lbnRzL0hlYWRlci5qc3g=) | `10.52% <0.00%> (-86.85%)` | :arrow_down: |
| [superset-frontend/src/components/IconTooltip.tsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2NvbXBvbmVudHMvSWNvblRvb2x0aXAudHN4) | `13.33% <0.00%> (-86.67%)` | :arrow_down: |
| [...rc/dashboard/components/gridComponents/Divider.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9jb21wb25lbnRzL2dyaWRDb21wb25lbnRzL0RpdmlkZXIuanN4) | `13.33% <0.00%> (-86.67%)` | :arrow_down: |
| [...end/src/SqlLab/components/ExploreResultsButton.jsx](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9jb21wb25lbnRzL0V4cGxvcmVSZXN1bHRzQnV0dG9uLmpzeA==) | `8.00% <0.00%> (-84.00%)` | :arrow_down: |
| ... and [401 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [b22e458...e2a2925](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (e2a2925) into [master](https://codecov.io/gh/apache/superset/commit/b22e45810866f874bf36b082bdb866cfe9614b40?el=desc) (b22e458) will **decrease** coverage by `3.53%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 66.85% 63.31% -3.54%
==========================================
Files 1015 486 -529
Lines 49575 29942 -19633
Branches 5079 0 -5079
==========================================
- Hits 33143 18959 -14184
+ Misses 16302 10983 -5319
+ Partials 130 0 -130
```
| Flag | Coverage Δ | |
|---|---|---|
| cypress | `?` | |
| javascript | `?` | |
| python | `63.31% <ø> (-0.83%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/sql\_validators/postgres.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvc3FsX3ZhbGlkYXRvcnMvcG9zdGdyZXMucHk=) | `50.00% <0.00%> (-50.00%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/mysql.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL215c3FsLnB5) | `79.59% <0.00%> (-12.25%)` | :arrow_down: |
| [superset/db\_engine\_specs/sqlite.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3NxbGl0ZS5weQ==) | `65.62% <0.00%> (-9.38%)` | :arrow_down: |
| [superset/databases/commands/create.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL2NyZWF0ZS5weQ==) | `83.67% <0.00%> (-8.17%)` | :arrow_down: |
| [superset/databases/commands/update.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL3VwZGF0ZS5weQ==) | `85.71% <0.00%> (-8.17%)` | :arrow_down: |
| [superset/connectors/sqla/models.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9zcWxhL21vZGVscy5weQ==) | `84.31% <0.00%> (-6.28%)` | :arrow_down: |
| [superset/databases/commands/test\_connection.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL3Rlc3RfY29ubmVjdGlvbi5weQ==) | `84.78% <0.00%> (-4.35%)` | :arrow_down: |
| [superset/utils/celery.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdXRpbHMvY2VsZXJ5LnB5) | `96.42% <0.00%> (-3.58%)` | :arrow_down: |
| ... and [541 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [b22e458...e2a2925](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] adam-stasiak commented on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
adam-stasiak commented on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759769805
@ktmud looks lite this:
![image](https://user-images.githubusercontent.com/25153919/104516027-f0c83a80-55f3-11eb-8a76-c358c360f333.png)
![image](https://user-images.githubusercontent.com/25153919/104516053-fcb3fc80-55f3-11eb-953c-3317dc225056.png)
looks fine to me.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (e2a2925) into [master](https://codecov.io/gh/apache/superset/commit/b22e45810866f874bf36b082bdb866cfe9614b40?el=desc) (b22e458) will **decrease** coverage by `3.03%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
- Coverage 66.85% 63.81% -3.04%
==========================================
Files 1015 486 -529
Lines 49575 29944 -19631
Branches 5079 0 -5079
==========================================
- Hits 33143 19110 -14033
+ Misses 16302 10834 -5468
+ Partials 130 0 -130
```
| Flag | Coverage Δ | |
|---|---|---|
| cypress | `?` | |
| javascript | `?` | |
| python | `63.81% <ø> (-0.33%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [superset/sql\_validators/postgres.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvc3FsX3ZhbGlkYXRvcnMvcG9zdGdyZXMucHk=) | `50.00% <0.00%> (-50.00%)` | :arrow_down: |
| [superset/db\_engine\_specs/mysql.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL215c3FsLnB5) | `79.59% <0.00%> (-12.25%)` | :arrow_down: |
| [superset/db\_engine\_specs/sqlite.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3NxbGl0ZS5weQ==) | `65.62% <0.00%> (-9.38%)` | :arrow_down: |
| [superset/databases/commands/create.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL2NyZWF0ZS5weQ==) | `83.67% <0.00%> (-8.17%)` | :arrow_down: |
| [superset/databases/commands/update.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL3VwZGF0ZS5weQ==) | `85.71% <0.00%> (-8.17%)` | :arrow_down: |
| [superset/databases/commands/test\_connection.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YWJhc2VzL2NvbW1hbmRzL3Rlc3RfY29ubmVjdGlvbi5weQ==) | `84.78% <0.00%> (-4.35%)` | :arrow_down: |
| [superset/utils/celery.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdXRpbHMvY2VsZXJ5LnB5) | `96.42% <0.00%> (-3.58%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `85.86% <0.00%> (-2.99%)` | :arrow_down: |
| [superset/views/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvY29yZS5weQ==) | `72.89% <0.00%> (-2.48%)` | :arrow_down: |
| [superset/datasets/api.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGF0YXNldHMvYXBpLnB5) | `89.49% <0.00%> (-1.83%)` | :arrow_down: |
| ... and [537 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [b22e458...e2a2925](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] junlincc merged pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
junlincc merged pull request #12498:
URL: https://github.com/apache/superset/pull/12498
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] agatapst edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
agatapst edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-760057175
@kkucharc regarding native filters:
- when Filter Bar is hidden, the sidebar should look the same in Dashboard page and Explore page and it does 👍
Explore sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 46](https://user-images.githubusercontent.com/47450693/104567842-9066e680-564f-11eb-8505-90c2a23d8b2e.png)
Dashboard sidebar:
![Zrzut ekranu 2021-01-14 o 09 59 35](https://user-images.githubusercontent.com/47450693/104567872-9957b800-564f-11eb-93a1-642bacbb03d2.png)
- when Filter Bar is open:
in your PR:
![Zrzut ekranu 2021-01-14 o 10 05 37](https://user-images.githubusercontent.com/47450693/104568321-3f0b2700-5650-11eb-8139-9ae6dcfc1f4b.png)
master:
![Zrzut ekranu 2021-01-14 o 10 00 04](https://user-images.githubusercontent.com/47450693/104568312-3a467300-5650-11eb-8677-69738669ac40.png)
I can see there is bigger distance between Filter Bar and Dashboard in your PR, but in my opinion both versions look just fine. If the cost of smaller distance between Native Filter Bar and Dashboard are UI issues in some other component, then I am totally fine with it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (5fdbd57) into [master](https://codecov.io/gh/apache/superset/commit/2d10a2c089d0d49c0efd4378b99864108c9afee2?el=desc) (2d10a2c) will **increase** coverage by `0.69%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
+ Coverage 62.66% 63.35% +0.69%
==========================================
Files 1014 486 -528
Lines 49593 29943 -19650
Branches 4839 0 -4839
==========================================
- Hits 31075 18970 -12105
+ Misses 18320 10973 -7347
+ Partials 198 0 -198
```
| Flag | Coverage Δ | |
|---|---|---|
| javascript | `?` | |
| python | `63.35% <ø> (-0.57%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/sql\_validators/postgres.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvc3FsX3ZhbGlkYXRvcnMvcG9zdGdyZXMucHk=) | `50.00% <0.00%> (-50.00%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/presto.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3ByZXN0by5weQ==) | `70.56% <0.00%> (-11.48%)` | :arrow_down: |
| [superset/db\_engine\_specs/sqlite.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3NxbGl0ZS5weQ==) | `65.62% <0.00%> (-9.38%)` | :arrow_down: |
| [superset/utils/celery.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdXRpbHMvY2VsZXJ5LnB5) | `96.42% <0.00%> (-3.58%)` | :arrow_down: |
| [superset/views/database/mixins.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvZGF0YWJhc2UvbWl4aW5zLnB5) | `80.70% <0.00%> (-1.76%)` | :arrow_down: |
| [superset/result\_set.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvcmVzdWx0X3NldC5weQ==) | `96.69% <0.00%> (-1.66%)` | :arrow_down: |
| [superset/connectors/sqla/models.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9zcWxhL21vZGVscy5weQ==) | `89.65% <0.00%> (-0.93%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `88.04% <0.00%> (-0.82%)` | :arrow_down: |
| ... and [540 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [d70db43...5fdbd57](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] codecov-io edited a comment on pull request #12498: fix: row component handler is visible
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #12498:
URL: https://github.com/apache/superset/pull/12498#issuecomment-759675838
# [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=h1) Report
> Merging [#12498](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=desc) (5fdbd57) into [master](https://codecov.io/gh/apache/superset/commit/2d10a2c089d0d49c0efd4378b99864108c9afee2?el=desc) (2d10a2c) will **increase** coverage by `0.75%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/superset/pull/12498/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #12498 +/- ##
==========================================
+ Coverage 62.66% 63.41% +0.75%
==========================================
Files 1014 486 -528
Lines 49593 29943 -19650
Branches 4839 0 -4839
==========================================
- Hits 31075 18987 -12088
+ Misses 18320 10956 -7364
+ Partials 198 0 -198
```
| Flag | Coverage Δ | |
|---|---|---|
| javascript | `?` | |
| python | `63.41% <ø> (-0.52%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [superset/db\_engines/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lcy9oaXZlLnB5) | `0.00% <0.00%> (-85.72%)` | :arrow_down: |
| [superset/db\_engine\_specs/hive.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2hpdmUucHk=) | `54.61% <0.00%> (-29.24%)` | :arrow_down: |
| [superset/db\_engine\_specs/presto.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3ByZXN0by5weQ==) | `70.56% <0.00%> (-11.48%)` | :arrow_down: |
| [superset/db\_engine\_specs/sqlite.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL3NxbGl0ZS5weQ==) | `65.62% <0.00%> (-9.38%)` | :arrow_down: |
| [superset/utils/celery.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdXRpbHMvY2VsZXJ5LnB5) | `96.42% <0.00%> (-3.58%)` | :arrow_down: |
| [superset/views/database/mixins.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvdmlld3MvZGF0YWJhc2UvbWl4aW5zLnB5) | `80.70% <0.00%> (-1.76%)` | :arrow_down: |
| [superset/result\_set.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvcmVzdWx0X3NldC5weQ==) | `96.69% <0.00%> (-1.66%)` | :arrow_down: |
| [superset/models/core.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvbW9kZWxzL2NvcmUucHk=) | `88.04% <0.00%> (-0.82%)` | :arrow_down: |
| [superset/db\_engine\_specs/base.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzL2Jhc2UucHk=) | `85.30% <0.00%> (-0.52%)` | :arrow_down: |
| [superset/reports/notifications/base.py](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree#diff-c3VwZXJzZXQvcmVwb3J0cy9ub3RpZmljYXRpb25zL2Jhc2UucHk=) | `95.00% <0.00%> (-0.46%)` | :arrow_down: |
| ... and [538 more](https://codecov.io/gh/apache/superset/pull/12498/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=footer). Last update [d70db43...5fdbd57](https://codecov.io/gh/apache/superset/pull/12498?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org