search

Protoqol / Prequel

Prequel for Laravel. Clear and concise database management. Unfortunately, not actively maintained due to time constraints.
https://packagist.org/packages/protoqol/prequel
MIT License
1.54k stars 96 forks source link

[FIX] Clearer horizontal table scrolling #5

Closed Protoqol closed 5 years ago

Protoqol commented 5 years ago

Currently it is not clear that you can scroll horizontally in the currently selected table.

Solutions might be

  1. Scroll bar at top instead of bottom
  2. Fade out on right side to indicate that there is more content
  3. Arrow pointing right with same purpose as 2.
James-N-M commented 5 years ago

Sounds like a good first issue I'll pick this one up if that's ok

Protoqol commented 5 years ago

Sounds good, good luck! If you have any questions you can just drop them here 😁

James-N-M commented 5 years ago

I gave it a go but I'm having some trouble setting up and working on the project. I forked the repo and tried to link to the local git repo from one of my laravel projects, and then made a simple change to the Welcome vue component. But the change isnt showing up. So basically my question is whats your process for setting up the project to develop it locally. Or do you have a tutorial link or documentation.

Protoqol commented 5 years ago

Hey, I have yet to make a proper contribution guide + testing resources. So here’s a shortened version. If the following is not working or is too much of a hassle, then wait a little bit until I get contribution all setup.

  1. Create laravel project or use existing one, as long as it has a valid database connection.
  2. Inside the project run composer require protoqol/prequel and navigate to vendors/protoqol/prequel. Then create a new git branch (using a descriptive name e.g. ‘fix-ui-horizontal-scrolling’)
  3. Run npm install
  4. (if making changes to php) Run composer install
  5. Take a look at the package.json, you’ll see that there are a bunch of laravel mix commands. It’s best to use npm run watch, this creates a local server with hot-reloading.
  6. When pushing changes, run npm run prod first to build!

Note: try, when possible, to use the commit naming conventions used by the repo’s commits. Not required, but very much appreciated!

Thanks for taking up the issue! I’m sorry for the lack of documentation and resources, I totally forgot while opening the issue! So if you’re still willing to take it, good luck! And else, no hard feelings!

QuintenJustus commented 5 years ago

Fixed by @James-N-M in #7