Skip to content

932509: Need to change the attractive ui for employee management application #16

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

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

Vignesh050801
Copy link
Collaborator

@Vignesh050801 Vignesh050801 commented Mar 12, 2025

Bug description

Need to change the company name and padding for sidebar in employee management.

Root cause

To improving the application ui for user friendly.

Reason for not identifying earlier

Find how it was missed in our earlier testing and development by analyzing the below checklist. This will help prevent similar mistakes in the future.

  • Guidelines/documents are not followed

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

  • Guidelines/documents are not given

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

Reason:

Guidelines/documents are not given - Requirement document

Action taken:

NA.

Related areas:

NA

Is it a breaking issue?

No

Solution description

Fixed the issue by changing the company name NexGen7 and overrides the padding-right value as 0px for sidebar.

Output screenshots

Before

image

After

image

Areas affected and ensured

NA

Ensured the following cases:

  • Ensured with all theme and bigger in Grid and pivot.

Additional checklist

This may vary for different teams or products. Check with your scrum masters.

  • Did you run the automation against your fix? - Yes

  • Is there any API name change? - No

  • Is there any existing behavior change of other features due to this code change? - No

  • Does your new code introduce new warnings or binding errors? - No

  • Does your code pass all FxCop and StyleCop rules? a- No

  • Did you record this case in the unit test or UI test? - No

Copy link

@rajapandiyan-settu rajapandiyan-settu left a comment

Choose a reason for hiding this comment

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

Fix fine.

Copy link
Collaborator

@jmfarook7 jmfarook7 left a comment

Choose a reason for hiding this comment

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

fix fine

@jmfarook7 jmfarook7 merged commit e4e9bc1 into master Mar 12, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants