Comments

pwolanin’s picture

This looks a reasonable addition that allows better customization.

In the last screenschot - I think I'd want the default text filled in rather than letting a blank be a fall-back to the default text. I guess that means you will potentially end up storing the default title in the DB?

An alternative would be to have an additioanl column listing the default values so the user can see what's going to be used if they leave it blank.

robloach’s picture

StatusFileSize
new3.35 KB

That's a good point. We don't want to store defaults in the database, but we do want the connection to be clear that it's an overriden name. After talking with you online, I think changing the column title to "Customized Name" would make that connection. Here's a reroll.

q0rban’s picture

Status: Needs review » Needs work
StatusFileSize
new175.78 KB

Hmm.. I'm torn about this. I want the end user UI to be dead simple. What about something like the attached? Ignore the icons and placement and all that.. this is just a mockup.

Basically every title would have an edit button next to it, and clicking that button hides the default title and shows the field for adding a custom title. Clicking the delete button would clear that field, hide it and show the default title.

robloach’s picture

I like that user interface a lot. Although the end user UI has to be dead simple, so does the development side. Remember that it has to also work without JavaScript or CSS enabled. Progress enhancement and feature scope are the key things here ;-) .

q0rban’s picture

> Although the end user UI has to be dead simple, so does the development side.

Agreed.

> Remember that it has to also work without JavaScript or CSS enabled.

If the textfield shows up underneath the default title with CSS/JS Disabled, I'm fine with that. It might be good to have a label for the textfield though with CSS/JS disabled, as it wouldn't be completely clear what that textfield would be for anymore.

If if you don't like that idea, the other option (although a little more complex and possibly error prone accross different browsers) is to leave it exactly as you have it in your screenshot and pull the table cell contents out of that column, merge with Column A, and remove Column C altogether ala my screenshot, all in the JS/CSS.

robloach’s picture

I think the best solution for now is to just get that title column in, and then make a follow up issue for the improved user interface you suggested.

robloach’s picture

Status: Needs work » Needs review
StatusFileSize
new4.5 KB

Here it is for the Drupal 5 branch. #2 is the patch for the Drupal 7 branch.

q0rban’s picture

Status: Needs review » Reviewed & tested by the community

Tested the update, the permissions and the form. Only tested #2. Looks good to me.

q0rban’s picture

Status: Reviewed & tested by the community » Patch (to be ported)
robloach’s picture

Version: 7.x-1.x-dev » 5.x-1.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new4.68 KB

Thanks for hitting that up.... Correction on the DRUPAL-5 patch....

robloach’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev
StatusFileSize
new3.46 KB

Here's DRUPAL-6--1.

q0rban’s picture

Hmm, I think I'd rather wait till we backport the other changes from HEAD back to D6 before this.

robloach’s picture

Assigned: robloach » Unassigned

Hit up DRUPAL-5: http://drupal.org/cvs?commit=310260 . The Drupal 6 branch is the only one that's missing, and then we can hit up #674960: Sexy title user interface. The DRUPAL-6--1 patch is at http://drupal.org/node/673798#comment-2438138 .

q0rban’s picture

q0rban’s picture

Status: Needs review » Fixed

Dang it.

robloach’s picture

Was this the only change missing in D6?!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

webankit’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

Ability to hide the title...

webankit’s picture

Status: Closed (fixed) » Active
guillaumev’s picture

Status: Active » Needs review
StatusFileSize
new1.63 KB

Attached is a patch which will allow you to hide the title, by entering in the customized name...

guillaumev’s picture

I just found out by reading the README file that it can also be hidden in the CSS by adding:

a.follow-link {
text-indent: -9999px;
}
Tezza’s picture

The result I get after applying the patch in #21 is a title that reads 'Follow [site name] on [customised name]'. I was expecting no title. What would be nice is 'Follow [customised name] on Twitter | FB' etc

robloach’s picture

Title: Follow Link Titles » Ability to hide the link title
Status: Needs review » Fixed

Thanks! Committed to both Drupal 6 and 7. Also took note of the problem Tezza ran into too.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.