Skip to content

Added icon support for JHtmlSidebar::addEntry#4178

Closed
manojLondhe wants to merge 2 commits intojoomla:stagingfrom
manojLondhe:staging
Closed

Added icon support for JHtmlSidebar::addEntry#4178
manojLondhe wants to merge 2 commits intojoomla:stagingfrom
manojLondhe:staging

Conversation

@manojLondhe
Copy link
Copy Markdown
Contributor

.

@peterlose
Copy link
Copy Markdown
Contributor

Great idea.

But when there are a lot of the same icons, for instance in com_users, the icons doesn't really serve any UX benefits:
skaermbillede 2014-08-25 kl 16 46 19

In com_content it's fine:
skaermbillede 2014-08-25 kl 16 49 08

@brianteeman
Copy link
Copy Markdown
Contributor

I'd much prefer we removed completely this stupid duplicated sidemenu

On 25 August 2014 15:50, Peter Lose notifications@github.com wrote:

Great idea.

But when there are a lot of the same icons, for instance in com_users, the
icons doesn't really serve any UX benefits:
[image: skaermbillede 2014-08-25 kl 16 46 19]
https://cloud.githubusercontent.com/assets/1738811/4031378/a1fd64a6-2c66-11e4-9994-a87abb748f76.png

In com_content it's fine:
[image: skaermbillede 2014-08-25 kl 16 49 08]
https://cloud.githubusercontent.com/assets/1738811/4031404/08121a8e-2c67-11e4-87ae-b3e795bd4cb4.png


Reply to this email directly or view it on GitHub
#4178 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

@peterlose
Copy link
Copy Markdown
Contributor

@brianteeman and only navigate through the top menu?

@brianteeman
Copy link
Copy Markdown
Contributor

yes. That way we have more space on the screen to display things we need to
display. See http://issues.joomla.org/tracker/joomla-cms/2316

On 25 August 2014 16:08, Peter Lose notifications@github.com wrote:

@brianteeman https://github.com/brianteeman and only navigate through
the top menu?


Reply to this email directly or view it on GitHub
#4178 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

@peterlose
Copy link
Copy Markdown
Contributor

I really don't agree with the proposed changes in that PR. By removing the left menu from, for instance com_content, you increase the navigation time and the table UX isn't improved.

@parthlawate
Copy link
Copy Markdown
Contributor

+1 to this.. Obviously the extension devs will need to make sure that they use different icons.. Also disagree on removing the left menu.. Its very handy to navigate.. Maybe we can have an option to collapse it so that you can have more screen space when you need it ?

@roland-d
Copy link
Copy Markdown
Contributor

Being able to collapse it and remember that it is collapsed (so on every page refresh it stays as it was) would be very nice.

@manojLondhe
Copy link
Copy Markdown
Contributor Author

collapsable
Yeah. Collapsable menu like Jomsocial as it can be seen in attached screenshot will be good, if it remembers that it is collapsed as @roland-d said.

@infograf768
Copy link
Copy Markdown
Member

@peterlose
Copy link
Copy Markdown
Contributor

The collapse idea in combination with some localstorage would be fine :)

@coolbung
Copy link
Copy Markdown

+1 for collapsible. That would be a fantastic addition.

@roland-d
Copy link
Copy Markdown
Contributor

I have added a proof-of-concept PR #4197 to add collapsible to the articles list.

@brianteeman
Copy link
Copy Markdown
Contributor

Now that the collapsible sidebar has been committed this PR no longer works. Are you able to look at the code again and update it


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4178.

@brianteeman
Copy link
Copy Markdown
Contributor

After over two months without response for the requested information I am closing this issue at this time. It can always be reopened if further information is provided as had been requested.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4178.

@brianteeman brianteeman closed this Mar 6, 2015
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.

8 participants