Skip to content

Commit

Permalink
Merge branch 'dev'
Browse files Browse the repository at this point in the history
  • Loading branch information
ddnexus committed Apr 12, 2024
2 parents 3846645 + 30e2d26 commit d7e3110
Show file tree
Hide file tree
Showing 23 changed files with 134 additions and 123 deletions.
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/Code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ body:
attributes:
label: 👀 Before submitting...
options:
- label: I upgraded to pagy version 8.1.1
- label: I upgraded to pagy version 8.1.2
required: true
- label: I searched through the [Documentation](https://ddnexus.github.io/pagy/)
required: true
Expand Down
7 changes: 3 additions & 4 deletions .github/latest_release_body.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
## ⚠ WARNING

We may drop pagy's less used CSS extras.<br>
We may drop pagy's less used CSS extras.
If you wish to keep your favorites alive, please, [vote here](https://github.com/ddnexus/pagy/discussions/categories/survey)

### ✴ What's new in 8.+ ✴
Expand All @@ -11,11 +11,10 @@ If you wish to keep your favorites alive, please, [vote here](https://github.com
- Better frontend helpers
- See the [CHANGELOG](https://ddnexus.github.io/pagy/changelog) for possible breaking changes

### Changes in 8.1.1
### Changes in 8.1.2

<!-- changes start -->
- Fixed broken aria-label for disabled links in Foundation (#685)
- Simplification of input variables and defaults: params and request_path are not instance variables
- Added "da" locale for aria_label.nav (closes #583)
<!-- changes end -->

[CHANGELOG](https://ddnexus.github.io/pagy/changelog)
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ If you upgrade from version `< 8.0.0` see the following:

<hr>

## Version 8.1.2

- Added "da" locale for aria_label.nav (closes #583)

## Version 8.1.1

- Fixed broken aria-label for disabled links in Foundation (#685)
Expand Down
2 changes: 1 addition & 1 deletion Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ GEM
listen (~> 3.0)
rexml (3.2.6)
rouge (4.2.1)
rubocop (1.63.0)
rubocop (1.63.1)
json (~> 2.3)
language_server-protocol (>= 3.17.0)
parallel (~> 1.10)
Expand Down
6 changes: 2 additions & 4 deletions README.md

Large diffs are not rendered by default.

8 changes: 4 additions & 4 deletions e2e/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@
},
"devDependencies": {
"@cypress/snapshot": "^2.1.7",
"@types/node": "20.12.4",
"@typescript-eslint/eslint-plugin": "7.5.0",
"@typescript-eslint/parser": "7.5.0",
"@types/node": "20.12.7",
"@typescript-eslint/eslint-plugin": "7.6.0",
"@typescript-eslint/parser": "7.6.0",
"cypress": "13.7.2",
"cypress-html-validate": "6.1.0",
"eslint": "8.57.0",
Expand All @@ -21,7 +21,7 @@
"eslint-plugin-promise": "6.1.1",
"html-validate": "8.18.1",
"start-server-and-test": "2.0.3",
"typescript": "5.4.4"
"typescript": "5.4.5"
},
"scripts": {
"lint-fix": "eslint --fix cypress --ext .ts",
Expand Down
2 changes: 1 addition & 1 deletion gem/apps/calendar.ru
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
# DOC
# https://ddnexus.github.io/pagy/playground/#4-calendar-app

VERSION = '8.1.1'
VERSION = '8.1.2'

require 'bundler/inline'
gemfile(true) do
Expand Down
2 changes: 1 addition & 1 deletion gem/apps/demo.ru
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
# DOC
# https://ddnexus.github.io/pagy/playground/#3-demo-app

VERSION = '8.1.1'
VERSION = '8.1.2'

require 'bundler/inline'
gemfile(true) do
Expand Down
2 changes: 1 addition & 1 deletion gem/apps/rails.ru
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# DOC
# https://ddnexus.github.io/pagy/playground/#2-rails-app

VERSION = '8.1.1'
VERSION = '8.1.2'

# Gemfile
require 'bundler/inline'
Expand Down
2 changes: 1 addition & 1 deletion gem/apps/repro.ru
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# DOC
# https://ddnexus.github.io/pagy/playground/#1-repro-app

VERSION = '8.1.1'
VERSION = '8.1.2'

require 'bundler/inline'
gemfile(true) do
Expand Down
2 changes: 1 addition & 1 deletion gem/bin/pagy
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/usr/bin/env ruby
# frozen_string_literal: true

VERSION = '8.1.1'
VERSION = '8.1.2'
APPS = %w[repro rails demo calendar].freeze

require_relative '../lib/optimist'
Expand Down
2 changes: 1 addition & 1 deletion gem/config/pagy.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# frozen_string_literal: true

# Pagy initializer file (8.1.1)
# Pagy initializer file (8.1.2)
# Customize only what you really need and notice that the core Pagy works also without any of the following lines.
# Should you just cherry pick part of this file, please maintain the require-order of the extras

Expand Down
4 changes: 2 additions & 2 deletions gem/javascripts/pagy-dev.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion gem/javascripts/pagy-module.js
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ const Pagy = (() => {
const trim = (a, param) => a.replace(new RegExp(`[?&]${param}=1\\b(?!&)|\\b${param}=1&`), "");
// Public interface
return {
version: "8.1.1",
version: "8.1.2",
// Scan for elements with a "data-pagy" attribute and call their init functions with the decoded args
init(arg) {
const target = arg instanceof Element ? arg : document;
Expand Down
2 changes: 1 addition & 1 deletion gem/javascripts/pagy.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion gem/lib/pagy.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

# Core class
class Pagy
VERSION = '8.1.1'
VERSION = '8.1.2'

# Gem root pathname to get the path of Pagy files stylesheets, javascripts, apps, locales, etc.
def self.root
Expand Down
2 changes: 1 addition & 1 deletion gem/lib/pagy/calendar.rb
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ def page_at(time, **opts)
def setup_unit_vars
raise VariableError.new(self, :format, 'to be a strftime format', @vars[:format]) unless @vars[:format].is_a?(String)
raise VariableError.new(self, :order, 'to be in [:asc, :desc]', @order) \
unless %i[asc desc].include?((@order = @vars[:order]))
unless %i[asc desc].include?(@order = @vars[:order])

@starting, @ending = @vars[:period]
raise VariableError.new(self, :period, 'to be a an Array of min and max TimeWithZone instances', @vars[:period]) \
Expand Down
8 changes: 3 additions & 5 deletions gem/locales/da.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,9 @@
da:
pagy:
aria_label:
# please add a comment in the https://github.com/ddnexus/pagy/issues/583
# posting the translation of the following "Page"/"Pages" with the plurals for this locale
nav: "Pages"
# one: ""
# other: ""
nav:
one: "Side"
other: "Sider"
prev: "Forrige"
next: "Næste"
prev: "&lt;"
Expand Down
9 changes: 7 additions & 2 deletions playground.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ $ pagy --help
```

!!!warning `bundle exec pagy`
Depending on your environment you may need to prepend `bundle exec` in all the examples shown in this page
Depending on your environment you may need to prepend `bundle exec` in all the examples shown.
!!!

### Pagy Apps
Expand Down Expand Up @@ -49,7 +49,12 @@ Edit it at will.

#### 2. Rails App

Starting point app to reproduce **rails related** pagy issues. Same usage as the [Repro App](#1-repro-app).
Starting point app to reproduce **rails related** pagy issues. It has the same usage as the [Repro App](#1-repro-app). i.e.:

```sh
pagy clone rails
pagy ./rails.ru
```

#### 3. Demo App

Expand Down
Loading

0 comments on commit d7e3110

Please sign in to comment.