Skip to content
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.

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

Feature: Save width of columns #14507

Open
diyoyo opened this issue Jul 21, 2018 · 16 comments · May be fixed by #16739
Open

Feature: Save width of columns #14507

diyoyo opened this issue Jul 21, 2018 · 16 comments · May be fixed by #16739
Labels
affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) enhancement A feature request for improving phpMyAdmin has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface

Comments

@diyoyo
Copy link

diyoyo commented Jul 21, 2018

Steps to reproduce

  1. Select a table to browse
  2. Adjust one of the column's width
  3. Come back on that very same page later

Expected behaviour

The width of the column should be set to the same as the latest modification

Actual behaviour

The width is back to default behavior.

Server configuration

Operating system:

Web server:

Database:

PHP version:

phpMyAdmin version:
4.8.2

Client configuration

Browser:
Firefox, Chrome
Operating system:

@lem9
Copy link
Contributor

lem9 commented Jul 23, 2018

Not a bug because the width is not supposed to be saved. Could be a feature request but this is open to discussion about what would be the best behavior.

@williamdes williamdes added the enhancement A feature request for improving phpMyAdmin label Nov 13, 2018
@williamdes williamdes changed the title Bug report: Width of columns are never saved Feature: Save width of columns Nov 13, 2018
@williamdes williamdes added the ui Issues relating to the user interface label Nov 13, 2018
@bozhi91
Copy link

bozhi91 commented May 12, 2019

Hey guys. Can I implement this feature?

@bozhi91
Copy link

bozhi91 commented May 14, 2019

williamdes, Can you assign me and my team to this feature. We are interested in implement it.
We need this for a school project.

@williamdes
Copy link
Member

@bozhi91 I can not assign you or your team since you are not phpmyadmin members but you are now "assigned" on this enhancement

Please keep us updated if you are not interested anymore or are actively working on this issue :)

@bozhi91
Copy link

bozhi91 commented May 14, 2019 via email

@williamdes
Copy link
Member

williamdes commented May 14, 2019 via email

@bozhi91
Copy link

bozhi91 commented May 14, 2019 via email

@williamdes
Copy link
Member

williamdes commented May 14, 2019 via email

@bozhi91
Copy link

bozhi91 commented Jun 10, 2019

Hi William,
I want to comment you something about the project.
We started the development and we decided to store the table's width in cookies instead of using another tables for saving all the data. This way could be more comfortable for the users because only the users using the same computer may see the changes. If someone connects remotely, will not see the changes because does not have the data stored in cookies.
Also will be more easy to implement this functionality because we will apply a minor changes in the code.
Hope this will not be a problem.

Regards,

@williamdes
Copy link
Member

Seems okay 👍
What do you think about that @ibennetch?

@bozhi91
Copy link

bozhi91 commented Jun 25, 2019

Hi William,
We have finished with our implementation. Now we need to do the pull request.
However, we were working on the forked project so how do I do the pull request?
Do I have to do it inside the forked project or what?
Thanks

@williamdes
Copy link
Member

Now we need to do the pull request.

Yes, please and add #14507 into the PR message

Open the PR from the fork to the phpMyAdmin repo.

@bozhi91
Copy link

bozhi91 commented Jun 25, 2019

What do you mean by: "add #14507 into the PR message"?
I have to comment that we have been working on this issue in the pull request message?

@williamdes
Copy link
Member

@bozhi91

We fixed the issue number: #14507

Nothing more is needed :)

@bozhi91
Copy link

bozhi91 commented Jun 26, 2019

Hi,
The pull request is done. I think you can see it.
We are waiting for your answer.
Thanks for your help.

shucon added a commit to shucon/phpmyadmin that referenced this issue Mar 16, 2021
Signed-off-by: Saksham Gupta <shucon01@gmail.com>
@shucon shucon linked a pull request Mar 16, 2021 that will close this issue
6 tasks
shucon added a commit to shucon/phpmyadmin that referenced this issue Jun 26, 2021
Signed-off-by: Saksham Gupta <shucon01@gmail.com>

Add functions and documentation

Signed-off-by: Saksham Gupta <shucon01@gmail.com>

Use db_table key as hash

Signed-off-by: Saksham Gupta <shucon01@gmail.com>
shucon added a commit to shucon/phpmyadmin that referenced this issue Aug 3, 2021
Signed-off-by: Saksham Gupta <shucon01@gmail.com>

Add functions and documentation

Signed-off-by: Saksham Gupta <shucon01@gmail.com>

Use db_table key as hash

Signed-off-by: Saksham Gupta <shucon01@gmail.com>

Fix code quality

Signed-off-by: Saksham Gupta <shucon01@gmail.com>
@yurenchen000
Copy link

yurenchen000 commented Dec 19, 2021

Hi, The pull request is done. I think you can see it. We are waiting for your answer. Thanks for your help.

seems master not have this feature.
I have use your diff in my deploy, and this make life easier.
// seems some time not store col width right: I resize colA localStore save colB or not store.
( I'm not sure is it related to hide OR re-orderd col. currenttly I edit localStore manually )


// the existing feature column sort & drag also help a lot !

👍 👍

@williamdes williamdes added this to the 5.2.0 milestone Dec 19, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Dec 19, 2021
KartikWatts added a commit to KartikWatts/phpmyadmin that referenced this issue Mar 26, 2022
Signed-off-by: KartikWatts <arorakartik4@gmail.com>
@ibennetch ibennetch removed this from the 5.2.0 milestone May 11, 2022
@ibennetch ibennetch added this to the 5.2.1 milestone May 11, 2022
@williamdes williamdes added the affects/5.2 This issue or pull-request affects 5.2.x releases (and maybe further versions) label Oct 25, 2022
@williamdes williamdes removed this from the 5.2.1 milestone Oct 25, 2022
@williamdes williamdes added affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) and removed affects/5.2 This issue or pull-request affects 5.2.x releases (and maybe further versions) labels Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) enhancement A feature request for improving phpMyAdmin has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants