Skip to content
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

Fixed: Fix the issue where the common Drawer Title is displayed in English #4673

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

fit2bot
Copy link
Contributor

@fit2bot fit2bot commented Feb 25, 2025

Fixed: Fix the issue where the common Drawer Title is displayed in English

@fit2bot fit2bot requested a review from a team February 25, 2025 03:32
}
title = toSentenceCase(action) + this.$t('WordSep') + toLowerCaseExcludeAbbr(title)
title = dispatchAction + this.$t('WordSep') + toLowerCaseExcludeAbbr(title)
return title
},
getDefaultDrawer(action) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code appears to be formatted correctly, but some changes have been made in the getDefaultTitle method of the component. These include:

  • Adding parentheses before and after the call expressions.
  • Replacing "export" with return keyword.

As far as potential issues or optimization suggestions go, I would need more specific details about what needs to be refined. It's generally best to analyze the overall structure and usage examples provided in order to make meaningful improvements. However, there might not necessarily be any significant structural modifications needed based on these small adjustments alone.

@ZhaoJiSen ZhaoJiSen merged commit c2b1763 into dev Feb 25, 2025
5 of 6 checks passed
@ZhaoJiSen ZhaoJiSen deleted the pr@dev@fix_drawer_common_title branch February 25, 2025 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants