Problem/Motivation

The current pattern of sample data under 'A CSV will be exported with this structure:' appears disorganized. We propose organizing the data in a tabular format for better clarity and structure.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#9 patch_html_remover.patch2.87 KBchancenyasulu
#4 issue.png26.94 KBtanuja bohra
#2 After .png293.33 KBenaznin
#2 beautify_sample_data_.patch1.84 KBenaznin
Screenshot 2023-09-18 at 12.46.26 PM.png249.94 KBenaznin

Issue fork path_redirect_import-3387868

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

enaznin created an issue. See original summary.

enaznin’s picture

enaznin commented
StatusFileSize
new beautify_sample_data_.patch1.84 KB
new After .png293.33 KB
enaznin’s picture

enaznin commented
Status: Active » Needs review
tanuja bohra’s picture

tanuja bohra commented
StatusFileSize
new issue.png26.94 KB

@enaznin ,
1. I can see one issue on mobile it's creating a white space. Can we use scroll on mobile devices?
2. Suggestion -> Can we use the same fonts (/admin/structure/types/manage/article/fields) and capitalize the first letter on column labels?

kevinquillen’s picture

kevinquillen commented
Status: Needs review » Needs work

This shouldn't be done by hardcoding in an HTML table. A render array using table for #theme should be used instead.

chancenyasulu made their first commit to this issue’s fork.

chancenyasulu opened merge request !22

chancenyasulu opened merge request !23

chancenyasulu’s picture

chancenyasulu commented
Status: Needs work » Needs review
StatusFileSize
new patch_html_remover.patch2.87 KB

Hello I have made changes regarding the comment #5 i have removed the the hard coded html to drupal way of doing it.

plopesc closed merge request !23

plopesc closed merge request !22

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Needs review » Reviewed & tested by the community

Thank you!

  • chancenyasulu authored 4c417076 on 2.0.x 
    Issue #3387868 by chancenyasulu, enaznin, Tanuja Bohra, kevinquillen,...
plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Reviewed & tested by the community » Fixed

Merged into 2.0.x. Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.