-
Notifications
You must be signed in to change notification settings - Fork 161
fix: Fix UI issues in rate limiting Resources page #1204
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
base: main
Are you sure you want to change the base?
fix: Fix UI issues in rate limiting Resources page #1204
Conversation
The blue colored text gave the false impression of clickable links, confusing users. The button was misaligned in the flex container. Changes: - Removed color='primary.main' from API Level Box - Removed cursor='pointer' from API Level Box - Removed color='primary.main' from Operation Level Box - Added variant, size, and sx alignment props to button Fixes wso2/api-manager#4383"
| </TextField> | ||
| {!isOperationRateLimiting && ( | ||
| <Button onClick={scrollToTop}> | ||
| <Button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Ratheshan03 Regarding this change, I have applied your fix locally and observed the behaviour. The button has gone a little down and the alignment looks a bit off from the bottom level compared with the input box. And also clicking on the button does not scroll to the top with the fix still. Will it be possible to check and fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @tharikaGitHub, Made the changes that you requested above, now the alignement is fixed more precisly and the functionality of the button is already seemed working and i also verified its working and scrolls to thte top. the evidence to the testing has been given below:
Publisher.-.WSO2.APIM.-.Google.Chrome.2025-10-30.14-27-46.mp4
Thanks.
|



Purpose
Fixed UI issues in the rate limiting section of the Publisher Portal Resources page where non-interactive text appeared as clickable links and a button was misaligned.
Root Cause
Issue 1: Misleading Blue Text
The "API Level" and "Operation Level" text elements had
color='primary.main'prop applied, making them appear in blue color. In WSO2 APIM portals, blue colored text consistently indicates clickable/interactive elements. Additionally, "API Level" hadcursor='pointer'which showed a hand cursor on hover, further reinforcing the false impression of clickability.Issue 2: Button Misalignment
The "Change rate limiting level ↑" button lacked proper flex alignment properties, causing it to be vertically misaligned with adjacent elements in the flex container. It also lacked appropriate size and style variants.
Solution
Removed the misleading styling from non-interactive text elements and added proper Material-UI styling props to the button for correct alignment and appearance.
Changes Made
Fixed "API Level" Text Styling
Fixed "Operation Level" Text Styling
Fixed Button Alignment, functionality and Styling
Code Changes
Change 1: API Level Text (Lines 161-173)
Change 2: Operation Level Text (Lines 200-206)
Change 3: Button Alignment (Lines 244-264)
Related Issues
Fixes UI issue when setting Rate limiting level in Resources page - wso2/api-manager#4383
Related PRs
N/A
Testing
Test Environment
Manual Test Round using Production build
✅ Test #1: API Level Text Color
✅ Test #2: Operation Level Text Color
✅ Test #3: Button Alignment & Functionality
Test Evidence
Screenshot 1: API Level Text Fix

"API Level" text now appears in normal color, not blue
Screenshot 2: Operation Level Text Fix

"Operation Level" text now appears in normal color, not blue
Screenshot 3: Button Alignment Fix

Button properly aligned with adjacent elements
Video: Button Functionality
Publisher.-.WSO2.APIM.-.Google.Chrome.mp4
Checklist
Note: Changes are purely CSS/styling modifications using Material-UI props. ESLint validation passed. Production build succeeded with no errors. All three fixes verified through manual testing with screenshots and video evidence.
Security Checks
Developer Checklist (Mandatory)
Note: No behavioral changes or migration impact. Pure UI styling improvements.