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

Attachment Type name #442

Closed
PaulBender opened this issue Apr 28, 2014 · 6 comments
Closed

Attachment Type name #442

PaulBender opened this issue Apr 28, 2014 · 6 comments
Assignees
Milestone

Comments

@PaulBender
Copy link
Member

@PaulBender PaulBender commented Apr 28, 2014

The attachment type table has a name column that stores the attachment name. However, there is no way of editing this name.

  • Add the ability to edit the attachment type name in Admin CP.
  • Add the attachment type name as a tooltip when hovering over an attachment icon.
@PaulBender PaulBender added this to the 1.8 Beta 1 milestone Apr 28, 2014
@PenguinPaul
Copy link
Contributor

@PenguinPaul PenguinPaul commented Apr 28, 2014

I would put this as a beta 2 target, any reason for beta 1 when it's an enhancement?

@PaulBender
Copy link
Member Author

@PaulBender PaulBender commented Apr 28, 2014

Oops, must've accidentally set that wrong.

@PaulBender PaulBender modified the milestones: 1.8 Beta 2, 1.8 Beta 1 Apr 28, 2014
@PenguinPaul PenguinPaul modified the milestones: 1.8 Beta 1, 1.8 Beta 2 Apr 28, 2014
@PaulBender PaulBender self-assigned this Apr 29, 2014
@PaulBender PaulBender modified the milestones: 1.8 Beta 1, 1.8 Beta 2 May 18, 2014
PaulBender added a commit that referenced this issue May 19, 2014
@PaulBender
Copy link
Member Author

@PaulBender PaulBender commented May 19, 2014

Finished

JordanMussi added a commit to JordanMussi/MyBB that referenced this issue May 24, 2014
@JordanMussi JordanMussi added the fixed label May 24, 2014
@DiogoParrinha
Copy link
Contributor

@DiogoParrinha DiogoParrinha commented May 27, 2014

I only have one complaint! On the ACP -> Attachment Types page, the icons need a tooltip as well :) @PaulBender

PaulBender added a commit that referenced this issue May 27, 2014
Added name tooltip to Admin CP page
@PaulBender
Copy link
Member Author

@PaulBender PaulBender commented May 27, 2014

Done!

@DiogoParrinha
Copy link
Contributor

@DiogoParrinha DiogoParrinha commented May 27, 2014

Good!

ATofighi added a commit to MyBBIr/mybb that referenced this issue May 29, 2014
Added name tooltip to Admin CP page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.