- Status changed to Needs review
almost 2 years ago 2:00pm 21 February 2023 - Status changed to Needs work
almost 2 years ago 2:02pm 22 February 2023 The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Needs review
over 1 year ago 1:35pm 27 March 2023 - Status changed to Needs work
over 1 year ago 1:29am 28 March 2023 - 🇳🇿New Zealand quietone
This issue is changing the user interface, so before and after screenshots are needed. Put them in the Issue summary or at least a link so that reviewers can see the latest work. See the accessibility core gate → .
And this will need a usability review → , adding tag.
The Issue Summary needs to be updated. The proposed resolution is to replace labels but does not say which labels are being replaced. And the remaining tasks should state what needs to be done.
I have not reviewed the patch.
- Status changed to Needs review
over 1 year ago 5:39am 28 March 2023 - 🇮🇳India rassoni Bangalore
Tested locally after applying #2 patch changes are reflected as per proposed resolution. The table has
module-list
class according to labels are replaced with Proposed resolution. Attached screenshot and video for reference.Steps to Test:
- Download #29 Patch locally and apply it.
- Clear cache.
- Navigate to structure -> view and edit any view which contains table format.
- Check in mobile view version, Lables are reflected as per Proposed resolution.
- First commit to issue fork.
- Status changed to Needs work
over 1 year ago 3:03pm 28 March 2023 - 🇺🇸United States smustgrave
Issue summary update has not happened.
@rckstr_rohan removing credit for the empty commit in #34.
- last update
over 1 year ago 29,402 pass