8000 Dialog: Add option to put the dialog title in a header element by rpkoller · Pull Request #2275 · jquery/jquery-ui · GitHub
[go: up one dir, main page]
Skip to content

Dialog: Add option to put the dialog title in a header element #2275

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.

Sign up for GitHub

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

Merged
merged 17 commits into from
Sep 9, 2024
Merged

Dialog: Add option to put the dialog title in a header element #2275

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
14dc459
Add option to set html tag for dialog modal title
rpkoller Jul 31, 2024
c22ba5b
Fix the line length lint error
rpkoller Aug 1, 2024
564b793
Add test to assert if the html tag is correctly set
rpkoller Aug 1, 2024
6f8307d
tagName is not a function. Removed parenthesis
rpkoller Aug 1, 2024
8f8c2a1
add the missing tab for the second line of uiDialogTitle
rpkoller Aug 1, 2024
a277562
move the <> to the option
rpkoller Aug 1, 2024
9238fc0
adjust to the changes in dialog.js and set the option to <h2>
rpkoller Aug 1, 2024
d4aa8a0
fix for the test
rpkoller Aug 2, 2024
b2f3e72
fix linting errors on the test
rpkoller Aug 2, 2024
7b9ea13
fix another linting error on the test
rpkoller Aug 2, 2024
478e434
add new option to common.js and common-deprecated.js
rpkoller Aug 2, 2024
140fa73
use and accept integer values only for html element
rpkoller Aug 5, 2024
f46ff12
Add new tests for dialog title heading levels
rpkoller Aug 5, 2024
6ffab2e
change to the new default value 0 for uiDialogTitleHeadingLevel
rpkoller Aug 5, 2024
5652776
fix linting errors
rpkoller Aug 5, 2024
c70786d
fix dialog title heading level tests
rpkoller Aug 5, 2024
8f583cb
add another assertion for the dialog title heading lvl with option foo
rpkoller Aug 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix for the test
  • Loading branch information
@rpkoller
rpkoller committed Aug 2, 2024
commit d4aa8a05b5ae105928a854c52e5c63beb8c0e44f
6 changes: 3 additions & 3 deletions tests/unit/dialog/core.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,11 +120,11 @@
QUnit.test( "ui dialog title tagname", function( assert ) {
assert.expect( 1 );

var element, wrapper;
var element, nodeName;

element = $( "<div>" ).dialog( { modal: true, uiDialogTitleTagName: "<h2>" } );
wrapper = document.querySelector( ".ui-dialog-title" );
assert.equal( wrapper.tagName, "h2", "The dialog title element is h2" );
nodeName = element.dialog( "widget" ).find( ".ui-dialog-title" ).get(0).nodeName.toLowerCase()

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery git 

There must be a space after this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery git 

There must be a space before this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery git 

Missing semicolon

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery stable 

There must be a space after this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery stable 

There must be a space before this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery stable 

Missing semicolon

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery git 

There must be a space after this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery git 

There must be a space before this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery git 

Missing semicolon

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery stable 

There must be a space after this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery stable 

There must be a space before this paren

Check failure on line 126 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery stable 

Missing semicolon
assert.equal (nodeName, "h2", "The dialog title element is h2" );

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery git 

Unexpected space between function name and paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery git 

There must be a space after this paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery stable 

Unexpected space between function name and paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / chrome | jQuery stable 

There must be a space after this paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery git 

Unexpected space between function name and paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery git 

There must be a space after this paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery stable 

Unexpected space between function name and paren

Check failure on line 127 in tests/unit/dialog/core.js

View workflow job for this annotation

GitHub Actions / firefox | jQuery stable 

There must be a space after this paren
} );

QUnit.test( "widget method", function( assert ) {
Expand Down
Loading
0