fix(Axis): xAxis axisTick custom interval funciton work error with boundaryGap false

[sz-p]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

Fixed issues

#20424

Details

Before: What was the problem?

After: How does it behave after the fixing?

Document Info

One of the following should be checked.

• This PR doesn't relate to document changes
• The document should be updated later
• The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

• This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

• Please squash the commits into a single one when merging.

Other information

[echarts-bot]

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

[github-actions]

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-20436@951540f

[Ovilia]

I think the correct position of the tick should be placed at the red position. Is that right? Run demo

[sz-p]

Thank you for reviewing!

I see the problem. and i found a new problem.

If every one tick in intervalList is true. the last one will be out of control.

The new fix way is change tick to right position and add last one after intervalList.

See the result after fix.

[Ovilia]

Thanks for the detailed test!

Can you explain why there is a tick before Thu when the last is false in the last two charts?

What's the current policy for the last tick to show?

[sz-p]

The current logic is simple: add last one after intervalList.

To form a tick pair, similar to the effect of boundaryGap: true, add an ending tick to the intervalList.

A potential issue arises if the intervalList is [true, false]. By adding an end tick, it becomes [true, true], which might seem odd.

A more intelligent way is if the tick is end of axis then add it. if not ignore it.

Should we use this?

[Ovilia]

I don't think axis ticks work in pairs, so I'd think add last one after intervalList works pretty intuitive to me.

[sz-p]

See new reault.

[Ovilia]

My expected behavior the the last tick to be show or not would be:

[true, true, true, true]: true
[false, false, false, false]: false
[true, false, false, false]: true
[true, false, true, true]: true
[true, false, true, false]: true
[false, true, true, true]: true

The logic is, the last one is not displayed only if none ticks is displayed. But I'm open to what you think. Please let me know your opinions on this and let's discuss about what's more intuitive.

[sz-p]

See new reault.

[sz-p]

Conflicts resolved.
Can you help me review it again. @Ovilia