Next #530
Merged
Next #530
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Updated `finance-quick-transfer-modal.tsx` to use `value` as a fallback if `amount` is not greater than 0. - Ensures accurate MMC amount is displayed in the transfer message.
- **`profile-header.tsx`**: Removed direct usage of `ProfileTransfer` and its transfer button for other user profiles. - **`profile-transfer.tsx`**: Introduced tooltip behavior and additional UI improvements for the transfer button. Enhanced interactivity with popovers and streamlined styling. - **`profile-toolbar.tsx`**: Re-integrated `ProfileTransfer` component for authenticated sessions in the updated toolbar context. These changes improve the usability and accessibility of profile transfer actions while refining the user interface and overall experience.
- Updated `finance-display-profile-info.tsx` to include a `fontSize` style of `1.3rem` for the profile display name. - Ensures improved readability for profile names in 'profile' mode.
- Introduced `trimPublicationContentExtraText` in `src/utils/text-transform.ts`. This utility trims text based on specific special characters. - Updated `publication-details-view.tsx` to utilize this trimming utility when rendering publication content with Markdown.
- Removed `ERRORS.BUNDLER_UNAVAILABLE` from `src/utils/notifications/errors.ts`. - Added `WARNING.BUNDLER_UNAVAILABLE` to `src/utils/notifications/warnings.ts` with updated message. - Replaced `notifyError` with `notifyWarning` for bundler unavailability in `src/hooks/use-account-session.ts`. This transition classifies the bundler unavailability as a warning, ensuring better contextual distinction for session-related notifications.
- Refactored `trimPublicationContentExtraText` to modularize logic: - Extracted `removeSpecialChars` function for removing predefined patterns. - Added `detectEmail` and `removeEmail` functions to handle email detection and removal. - Updated logic to ensure processed text ends with a period. - Improved readability and maintainability by splitting responsibilities.
…mprovement # Conflicts: # src/sections/finance/components/finance-display-profile-info.tsx # src/sections/finance/components/finance-quick-transfer-modal.tsx # src/sections/publication/view/publication-details-view.tsx
- Added a detailed `emailRegex` with extended documentation in `text-transform.ts` to centralize email validation logic. - Updated `detectEmail` and `removeEmail` functions to reuse the new `emailRegex` constant, improving maintainability and consistency.
- Updated `emailRegex` in `src/utils/text-transform.ts` to include word boundary markers (`\b`) at both ends. - This change ensures more accurate email matching and prevents partial matches in strings.
- Updated `text-transform.ts` to replace `charAt` and length check with `endsWith` for verifying the presence of a period. - This improves code readability and leverages a more intuitive string method.
…ns-send-quick-improvement refactor: ui and texts minor enhancements
fix: update metadata
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.