plainheart commented on code in PR #20129:
URL: https://github.com/apache/echarts/pull/20129#discussion_r1674150528
##########
src/component/legend/legendAction.ts:
##########
@@ -17,59 +17,81 @@
* under the License.
*/
-// @ts-nocheck
+import {curry, each, hasOwn} from 'zrender/src/core/util';
+import { EChartsExtensionInstallRegisters } from '../../extension';
+import { Payload } from '../../util/types';
+import type GlobalModel from '../../model/Global';
+import type LegendModel from './LegendModel';
-import {curry, each} from 'zrender/src/core/util';
+type LegendSelectMethodNames = 'select' | 'unSelect' | 'toggleSelected' |
'allSelect' | 'inverseSelect';
-function legendSelectActionHandler(methodName, payload, ecModel) {
- const selectedMap = {};
- const isToggleSelect = methodName === 'toggleSelected';
- let isSelected;
- // Update all legend components
- ecModel.eachComponent('legend', function (legendModel) {
- if (isToggleSelect && isSelected != null) {
- // Force other legend has same selected status
- // Or the first is toggled to true and other are toggled to false
- // In the case one legend has some item unSelected in option. And
if other legend
- // doesn't has the item, they will assume it is selected.
- legendModel[isSelected ? 'select' : 'unSelect'](payload.name);
- }
- else if (methodName === 'allSelect' || methodName === 'inverseSelect')
{
+function legendSelectActionHandler(methodName: LegendSelectMethodNames,
payload: Payload, ecModel: GlobalModel) {
+ const isAllSelect = methodName === 'allSelect' || methodName ===
'inverseSelect';
+ const selectedMap: Record<string, boolean> = {};
+
+ const actionLegendIndices: number[] = [];
+ ecModel.eachComponent({ mainType: 'legend', query: payload }, function
(legendModel: LegendModel) {
+ if (isAllSelect) {
legendModel[methodName]();
}
else {
legendModel[methodName](payload.name);
- isSelected = legendModel.isSelected(payload.name);
}
- const legendData = legendModel.getData();
- each(legendData, function (model) {
- const name = model.get('name');
- // Wrap element
- if (name === '\n' || name === '') {
- return;
- }
- const isItemSelected = legendModel.isSelected(name);
- if (selectedMap.hasOwnProperty(name)) {
- // Unselected if any legend is unselected
- selectedMap[name] = selectedMap[name] && isItemSelected;
- }
- else {
- selectedMap[name] = isItemSelected;
- }
+
+ makeSelectedMap(legendModel, selectedMap);
+
+ actionLegendIndices.push(legendModel.componentIndex);
+ });
+
+ const allSelectedMap: Record<string, boolean> = {};
+
+ // make selectedMap from all legend components
+ ecModel.eachComponent('legend', function (legendModel: LegendModel) {
+ each(selectedMap, function (isSelected, name) {
+ // Force other legend has same selected status
+ // Or the first is toggled to true and other are toggled to false
+ // In the case one legend has some item unSelected in option. And
if other legend
+ // doesn't has the item, they will assume it is selected.
+ legendModel[isSelected ? 'select' : 'unSelect'](name);
});
+
+ makeSelectedMap(legendModel, allSelectedMap);
});
+
// Return the event explicitly
- return (methodName === 'allSelect' || methodName === 'inverseSelect')
+ return isAllSelect
? {
- selected: selectedMap
+ selected: selectedMap,
+ // return legendIndex array to tell the developers which legends
are allSelect / inverseSelect
+ legendIndex: actionLegendIndices
Review Comment:
文档 PR 合了之后会改，因为一直以来都没有，故暂只考虑给全选加，后面可以按需再完善。
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
