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

Graphical Improvments #58

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

paspiz85
Copy link

My changes are:

  • Fixed position in bug view page
  • Fixed margins in view page and user page
  • Changed sidebar icon

Pasquale Pizzuti and others added 2 commits April 17, 2024 18:36
@brlumen
Copy link
Member

brlumen commented Apr 18, 2024

@paspiz85, thank you for your contribution to the development of the project!
Could you please attach screenshots visualizing the changes you are making?

@paspiz85
Copy link
Author

Current-week
View-Issue

@brlumen
Copy link
Member

brlumen commented Apr 18, 2024

About moving the calendar display area to a separate block:
Since this is a significant change in the positioning of information on the page, the only acceptable option for such a change is to implement it as a customizable option. The user or administrator, globally or personally, should be able to select the current display option or the one you offer. The default should be the current option, as an alternative, yours.

That's the only option I'm willing to consider for your PR.

@paspiz85
Copy link
Author

0009309-dfsgf-MantisBT

@paspiz85
Copy link
Author

I completely agree that it would be appropriate to add a configuration to allow both display modes. I think it's important to provide the option for this display since the calendar information is secondary to the Summary and Description, which might not be visible on sufficiently large screens (see the last screenshot). At the moment, I can't contribute further, but I hope someone continues my work. Thank you for your time.

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.

None yet

2 participants