Reviewing merge request #5: Show the bookmark title in the icon's palette

I thought it is a bit weird that to see a bookmark title in read you have to:

1) Click to go to the bookmark
2) Hover over the bookmark ribbon (moving your mouse 1/2 way across the screen)

It is very annoying as it creates unnecessary friction between user and activity.

I think it would be better to show them in the tool tip for the icons in the bookmark list on the bottom of the screen. Is there a reason we don't currently?

Here is a picture: http://oi61.tinypic.com/2cqe0k1.jpg

Requested by: samdroid
Status: Merged
Source repository: samdroids-mainline:title-in-palette
Target repository: mainline:master
Created at: June 14th, 2014 11:37

Commits that would be merged:

Version 2

Version 1
Version 2

4b12edd
b14ced4

Show the bookmark title in the icon's palette

Showing

Showing 4b12edd-b14ced4

Comments
Include comments entered inline

Pushed new version 1

samdroid | 2014-06-14 11:37:22 UTC

Pushed new version 2

samdroid | 2014-06-14 11:44:56 UTC

→ State changed from Open to Merged

Gonzalo Odiard | 2014-07-07 14:08:26 UTC

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository

One way to merge in these changes is to pull them into a local branch for review, verifying the changes and then merge to back to master.

# Check out a new branch for integration
git checkout -b merge-requests/5

# Fetch the merge request into this branch
git pull \
  git://git.sugarlabs.org/read/mainline.git \
  refs/merge-requests/5

# Show the commits, assess they are okay
git log --pretty=oneline --abbrev-commit master..merge-requests/5

# To apply the changes to your branch:
git checkout master
git merge merge-requests/5
git push origin master

Reviewing merge request #5: Show the bookmark title in the icon's palette

Commits that would be merged:

Comments Include comments entered inline

Add a new comment:

Gitorious

Comments
Include comments entered inline