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

Indexes shown at both tbl_structure.php and tbl_relation.php #12961

Closed
OlafvdSpek opened this issue Feb 6, 2017 · 24 comments · Fixed by #16349
Closed

Indexes shown at both tbl_structure.php and tbl_relation.php #12961

OlafvdSpek opened this issue Feb 6, 2017 · 24 comments · Fixed by #16349
Assignees
Labels
enhancement A feature request for improving phpMyAdmin good first issue hacktoberfest newbie ui Issues relating to the user interface
Milestone

Comments

@OlafvdSpek
Copy link

Shouldn't it be on only one page?

@mynetx
Copy link
Contributor

mynetx commented Feb 10, 2017

Thanks for noting this. Which page would you prefer as per logical reasoning?

@OlafvdSpek
Copy link
Author

Structure

@madhuracj
Copy link
Contributor

I believe this was intentional. See #6021

@OlafvdSpek
Copy link
Author

I'm not sure why automatically creating required indexes requires showing the indexes as well.

@madhuracj
Copy link
Contributor

If you read the full thread you'll note that the idea of adding indexes automatically has been abandoned and allowing the user to create indexes on the relations page has been implemented instead.

@OlafvdSpek
Copy link
Author

OlafvdSpek commented Feb 11, 2017

I did read the full page but apart from one summary update I didn't see that being mentioned..

Perhaps both pages could be merged?
It'd be nice to having a single overview of all stuff of a table.

@madhuracj
Copy link
Contributor

I feel the page will be way too long if we merge the two.

@OlafvdSpek
Copy link
Author

Would it?
I think the Internal relationships could easily be shown in the main table structure table. An action button to edit the internal relationship for a field would be simple as well..

@OlafvdSpek
Copy link
Author

and allowing the user to create indexes on the relations page has been implemented instead.

Foreign keys require indexes on the target table, but this functionality applies to the source table. So it's useless.

@nijel
Copy link
Contributor

nijel commented Feb 14, 2017

Looking at the structure page, it might be better to split it up more in second level tabs:

  • structure
  • indexes
  • partitions
  • foreign keys

That would probably be more consistent than current approach. What do you think?

@nijel nijel added enhancement A feature request for improving phpMyAdmin newbie ui Issues relating to the user interface labels Feb 14, 2017
@OlafvdSpek
Copy link
Author

I think fewer pages and less second-level navigation is better.

I see no reason to split structure and indexes..

@nijel
Copy link
Contributor

nijel commented Feb 20, 2017

Clearly current state is inconsistent (indexes are show on both structure and foreign keys, partitions are shown collapsed at structure page), what really doesn't look good. I thought using consistently second level navigation for all these table structure features might be better.

@OlafvdSpek
Copy link
Author

Dropping indexes from Relation view seems like a trivial fix.

For table structure, IMO partitions is taking up too much space when partitions isn't used at all.
"Table comments: " is showing while there is no comment.
Adding columns should be a single button / link, same for adding indexes.
Print, propose table structure, track table and improve table structure could perhaps be somewhere else too. Perhaps Improve should just be called Normalize.
Like the little key icon, I'd like to see an icon for columns with foreign keys and relations.

@ibennetch
Copy link
Member

ibennetch commented Feb 20, 2017

About the comment being shown when there is not comment, see #13025 for a proposed fix.

@ibennetch
Copy link
Member

ibennetch commented Feb 20, 2017

I'm giving the new Github Projects feature a try with this as a way to collect all the ideas in one place that's easy to track: https://github.com/phpmyadmin/phpmyadmin/projects/1

What do you all think?

@nijel
Copy link
Contributor

nijel commented Feb 20, 2017

I've never got used to this kind of project management, but let's try it :-).

@ibennetch
Copy link
Member

As far as changing the text of the link from "Improve table structure" to "Normalize," I think that needs to happen in master because we really should be in a string freeze for 4.7.

@ibennetch
Copy link
Member

The text change to normalize has been complete for master and will be part of the next major release (#13070).

@alexa-ai
Copy link

Is this still open? Based on #13070, it looks like it's resolved.

@williamdes
Copy link
Member

I agree, do you @ibennetch?

@OlafvdSpek
Copy link
Author

#13070 only renamed a button, so I don't think you can base anything on that.

@ibennetch
Copy link
Member

I agree with Olaf, this hasn’t really been resolved.

@williamdes williamdes added this to Triage zone in Enhancements Jan 23, 2020
@petrduda
Copy link
Contributor

Hello @williamdes, I would like to fix this, is the target branch master or QA_5_0, please?

@williamdes
Copy link
Member

Hello @williamdes, I would like to fix this, is the target branch master or QA_5_0, please?

Hello @petrduda

Since it is an enhancement you can use master branch instead of QA_5_0

petrduda added a commit to petrduda/phpmyadmin that referenced this issue Sep 13, 2020
…tion

Signed-off-by: Petr Duda <petrduda@seznam.cz>
williamdes added a commit that referenced this issue Sep 20, 2020
…bl_relation

Pull-request: #16349
Fixes: #12961

Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes added a commit that referenced this issue Sep 20, 2020
Signed-off-by: William Desportes <williamdes@wdes.fr>
Enhancements automation moved this from Triage zone to Done Sep 20, 2020
@williamdes williamdes self-assigned this Sep 20, 2020
@williamdes williamdes added this to the 5.1.0 milestone Sep 20, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement A feature request for improving phpMyAdmin good first issue hacktoberfest newbie ui Issues relating to the user interface
Projects
Enhancements
  
Done
Development

Successfully merging a pull request may close this issue.

9 participants