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

Feature/improve design of front page #287

Merged
merged 7 commits into from
Jun 3, 2024

Conversation

irinahpe
Copy link
Contributor

Added tables to the main ratingallocate page in order to make the buttons and status information more readable.

Copy link
Contributor

@Laur0r Laur0r left a comment

Choose a reason for hiding this comment

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

Thanks, it looks already much better! Just some minor remarks

locallib.php Show resolved Hide resolved
renderer.php Show resolved Hide resolved
renderer.php Show resolved Hide resolved
$this->page->url,
array('action' => ACTION_DISTRIBUTE_UNALLOCATED_EQUALLY)
);
$buttondisteq = new single_button($distributeunallocatedurleq,
Copy link
Contributor

Choose a reason for hiding this comment

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

The buttons are very large for the German version. The string is longer but it seems the button size is larger than it has to be. Can you have a look at that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it is not possible to make the buttons smaller manually, since this is just how they look like in this theme with a word wrap. I could give them a fixed size, but that wouldn't look good on different sized windows of the page.

styles.css Outdated

.ratingallocate_front_page_buttons {
margin: auto;
width: fit-content;
Copy link
Contributor

Choose a reason for hiding this comment

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

This prohibits the alignment of the 'distribute unallocated' buttons

renderer.php Show resolved Hide resolved
Copy link
Contributor

@Laur0r Laur0r left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

@Laur0r Laur0r merged commit 66f9eb1 into master Jun 3, 2024
38 checks passed
@Laur0r Laur0r deleted the feature/improve_design_of_front_page branch June 3, 2024 08:46
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.

2 participants