Project

General

Profile

Actions

Feature #27346

closed

Use the new pagination style for the calendars view

Added by Bernhard Rohloff over 6 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Calendar
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:

Description

This patch converts the previous and next month links in the calendars view into buttons in the new pagination style. It also aligns the buttons vertically with the input fields on the left.


Files


Related issues

Related to Redmine - Patch #21258: Use <ul> to do pagination, styling in a GitHub like manner with improved handling in responsive modeClosedJean-Philippe Lang

Actions
Actions #1

Updated by Bernhard Rohloff over 6 years ago

This is the result of the patch...



Actions #2

Updated by Go MAEDA over 6 years ago

  • Subject changed from Use the new pagination style for the calendars view. to Use the new pagination style for the calendars view
  • Category changed from UI to Calendar
  • Target version set to 4.1.0

It looks nice and improves the consistency of UI. I am setting target version to 4.1.0.
Thank you for posting the patch.

Actions #3

Updated by Marius BĂLTEANU over 6 years ago

  • Related to Patch #21258: Use <ul> to do pagination, styling in a GitHub like manner with improved handling in responsive mode added
Actions #4

Updated by Marius BĂLTEANU over 5 years ago

This is another small change that can be included in 4.0.0.

Actions #5

Updated by Go MAEDA over 5 years ago

Marius BALTEANU wrote:

This is another small change that can be included in 4.0.0.

I have not committed this patch because this patch changes a style ".buttons". The change influences all pages which use the class, not only the calendar page.

Actions #6

Updated by Marius BĂLTEANU over 5 years ago

Go MAEDA wrote:

Marius BALTEANU wrote:

This is another small change that can be included in 4.0.0.

I have not committed this patch because this patch changes a style ".buttons". The change influences all pages which use the class, not only the calendar page.

Oh, I need to admit that I didn't see that change.

Actions #7

Updated by Bernhard Rohloff over 5 years ago

I'm currently on the road but I can have a look at the patch on Sunday.
At a first glance I think the new styles came into the patch by accident.

Actions #8

Updated by Bernhard Rohloff over 5 years ago

Ok, I've looked at the patch and the style changes for the 'buttons' class are necessary to geht the new buttons for previous and next month in line with the form controls on the left.
After nearly a year passed by since I've submitted this patch I don't think it's the right solution anymore and I'll submit a better one for 4.1.0.

Actions #9

Updated by Go MAEDA almost 5 years ago

  • Target version changed from 4.1.0 to Candidate for next major release
Actions #10

Updated by Go MAEDA 12 months ago

Go MAEDA wrote in #note-5:

I have not committed this patch because this patch changes a style ".buttons". The change influences all pages which use the class, not only the calendar page.

I slightly updated the patch. The new patch addresses the above point by setting margin-top against .controller-calendars p.buttons instead of p.buttons.

Setting the target version to 5.1.0.

Actions #11

Updated by Go MAEDA 12 months ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch in r22235. Thank you.

Actions #12

Updated by Go MAEDA 12 months ago

  • Tracker changed from Patch to Feature
Actions

Also available in: Atom PDF