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

Update README.md #401

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #401

wants to merge 1 commit into from

Conversation

dibydrams
Copy link

Sorting compatible devices in alphabetical order.

Sorting compatible devices in alphabetical order.
@XAVERRenogadeAngle
Copy link

Do people just want to post so they feel like a part of something.

@dibydrams
Copy link
Author

dibydrams commented Feb 4, 2017

No need to be condescending if you do not appreciate the contribution even if minimal. I just found that more convenient to read for people. Frankly I don't care about being part of this.

@nullpixel
Copy link
Contributor

The devices are storted in order of release

@dibydrams
Copy link
Author

If so, the iPod touch 6G would be before the iPad Air 2 and the iPad mini 4 right? Anyway is this sorting really relevant now at this place? Isn't it more convenient for people to read in alphabetical order? It was just a proposal.

@ezhes
Copy link

ezhes commented Feb 5, 2017

This is such a useless pull. I get that you want to be on the contributor list for a jailbreak but come on.

@dibydrams
Copy link
Author

What's your problem here? I'm an editor and I usually pay attention to this kind of detail, it was just to help a little as it's an open source project. You can take changes directly without merging my PR honestly I don't give a damn. I agree this is the least important part of the project for you, but men no need to be condescending.

@XAVERRenogadeAngle
Copy link

Close this you buffoon.
Sorry let me put it in alphebitacle order so people don't get confused by reading this.
Buffoon Close This You.

@dibydrams
Copy link
Author

@XAVERRenogadeAngle In fact you're the one trying to give you importance by criticizing the PRs relative to the Readme file, the only thing you understand pretty much in this project. And besides, you're rude.

@XAVERRenogadeAngle
Copy link

Ok here is my point. You can organize the device list in plenty of ways.
Alphabetical
Order of compatibility
Order released
By the size
By screen size
Processing power
All iPhones iPads etc in categories to themsleves
And you can organize those sub categories in these ways as well. But then In what order would you present the categories.
Basically the list is endless
Nobody is going to care what order they are in. So I feel that this should be closed because posting is only for code issues and this one is clutter.

@dibydrams
Copy link
Author

Certainly there is a lot of different ways for organizing this list, and I'm just suggesting the one that is the most logical and comfortable to read for someone trying to find out if its own devices are compatible or not.

When you look for supporting devices anywhere else for applications, softwares, etc you never find them organized by released date, screen size or whatever. It's just common sense but it seems you take pleasure to epilogue on the subject.

It's your comments that clutter up the discussion on a very very simple proposition that doesn't even need one. It's sad to waste its time on things so futile. Personally I have better things to do elsewhere. If the admin appreciate the PR, he merges it, if not, he doesn't. End.

@Sn0wFr1tz
Copy link

Oh my god yall are rude as hell its just an idea and if you finna be a dickhead about it get your bubble dum gum ass outta here and I'm not @dibydrams side because if he merges it he merges it if he doesn't he doesn't and it's just an idea SO FUCKING CALM YOURSELF PLEASE DO THE WORLD A FAVOR AND CALM YOUR SELF...

@akachronix
Copy link

@Sn0wFr1tz dude... no one CARES

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

Successfully merging this pull request may close these issues.

6 participants