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

Add drop-down with available databases for Move operation #46

Open
eugef opened this issue Feb 11, 2014 · 6 comments
Open

Add drop-down with available databases for Move operation #46

eugef opened this issue Feb 11, 2014 · 6 comments

Comments

@eugef
Copy link
Contributor

eugef commented Feb 11, 2014

We have "Move" and "Move all" functionality for the keys.

I think it would be nice to have drop-down with available databases here instead of plain text field
phpredmin move page
phpredmin move popup

@sasanrose
Copy link
Owner

Good idea

@blitzmann
Copy link
Contributor

This was one of my other purposes for setting max_databases. Never got around to doing it tho. I can get started on this once pull request for new modals #47 is resolved.

@blitzmann
Copy link
Contributor

Actually... You could move data to a database that has not been initialized yet. So really you can enter any number that is not above max_databases. Moving a key to a DB is one way I used when initializing my database through PHPRedmin before I added a Add DB thing...

Would a dropdown be good for this (I know we kinda discussed it and it's relasion to large number of databases before)? Or perhaps a radio button to choose between current DB and new DB, and change the dropdown options from there? Or a simple message saying what the max_databases is and to not select a integer higher than that?

@eugef
Copy link
Contributor Author

eugef commented Feb 12, 2014

@blitzmann, you can't move key to the same DB, only to another

@blitzmann
Copy link
Contributor

True, I overlooked that. Still, you can move the key to any DB that is not the current and that is included in max_databases. Just need to decide to to present that information.

@sasanrose
Copy link
Owner

Sorry guys. I had to take an IELTS exam. Thus, I needed to study :) Anyway, I believe it would be a better idea to just have a text input with validation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants