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

user Interface improvements #63

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

hpnightowl
Copy link

♻️ Current situation

Describe the current situation. Explain current problems, if there are any. Be as descriptive as possible (e.g., including examples or code snippets).

  • Removed hardcoded strings that will be good for later future development as well in translations
  • searching perspective
  • layout overlapping

💡 Proposed solution

  • shuffled the layout for better searching ability (reason mentioned in commit hpnightowl@a22908b
  • added hardcoded strings for logins
  • improved the layout added margins

Describe the proposed solution and changes. How does it affect the project? How does it affect the internal structure (e.g., refactorings)?

📷 Screenshots

image

Provide screenshots showcasing the changes before and after the proposed changes (if applicable).

📚 Release Notes

Provide a summary of the changes or features from a user's point of view. If there are breaking changes, provide migration guides using code examples of the affected features.

📝 Testing

Which tests were added? Which existing tests were adapted/changed? Which situations are covered, and what edge cases are missing?

- can be helpful later in traslations as well
- it should follow userpreference font size
- reason behind it is simple like a normal person we will see first what
  site the 2fa is needed for then we go the email for which it is needed
  then we go with the 2fa. from top hierarchy prespective.
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.

1 participant