SAK-51080 Duplicate Site: New parameter for title #13377
Conversation
![accesslint[bot]](https://avatars.githubusercontent.com/in/1502?s=60&v=4){kind=small}
| <label for="id"> | ||
| $tlang.getString("sittoo.tit") | ||
| </label> | ||
| <input type="text" name="title" id="title" /> |
There was a problem hiding this comment.
Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.
There was a problem hiding this comment.
Now that accesslint mentions it, you can also add the same for the id field, thanks!
| } | ||
| catch (IdUsedException e) | ||
| { | ||
| addAlert(state, rb.getFormattedMessage("sitact.thesitid", new Object[]{id})); | ||
| return; | ||
| } | ||
| catch (IdUnusedException e) | ||
| { | ||
| addAlert(state, rb.getFormattedMessage("sitact.thesitid", new Object[]{id})); |
There was a problem hiding this comment.
| addAlert(state, rb.getFormattedMessage("sitact.thesitid", new Object[]{id})); | |
| addAlert(state, rb.getFormattedMessage("sitact.notfound", new Object[]{id})); |
| <label for="id"> | ||
| $tlang.getString("sittoo.tit") | ||
| </label> | ||
| <input type="text" name="title" id="title" /> |
There was a problem hiding this comment.
Now that accesslint mentions it, you can also add the same for the id field, thanks!
|
There's one cypress test for announcements failing, but it shouldn't be related. Any clues @ottenhoff ? |
Maybe this one: #13371... |
bgarciaentornos
changed the title
|
The problem is with the new announcements cypress test. I will fix it and re-run the test. |
SAK-51080