Ticket #1514 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

mcedit: optional different background for columns > 80

Reported by: sergey-feo Owned by: angel_il
Priority: major Milestone: 4.7.1
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master

Description

In programming good style is do not exceed 80 columns.
It will be good if mcedit will have option for different background color for columns 81 and righter.

You can see examples in attached files.

Attachments

80qtc.gif (128.3 KB) - added by sergey-feo 9 years ago.
qt designer with 80 columns delimiter
80.gif (96.6 KB) - added by sergey-feo 9 years ago.
How can it looks in mcedit

Change History

Changed 9 years ago by sergey-feo

qt designer with 80 columns delimiter

Changed 9 years ago by sergey-feo

How can it looks in mcedit

comment:1 in reply to: ↑ description Changed 9 years ago by angel_il

Replying to sergey-feo:

How can it looks in mcedit

are you really need this?

comment:2 Changed 9 years ago by angel_il

  • Owner set to angel_il
  • Status changed from new to accepted

comment:3 Changed 9 years ago by angel_il

  • Milestone changed from future releases to 4.7

comment:4 Changed 9 years ago by sergey-feo

are you really need this?

I can live without this, but with this feature life is better :-)
Please note that old Borland C++ Builder 6 also have feature like this.

comment:5 Changed 9 years ago by angel_il

  • Milestone changed from 4.7 to 4.7.0-pre4

comment:6 Changed 9 years ago by NNemec

I agree that this would be a really helpful feature. I am working on a project where the 80char limit is strict policy. I don't know how often a simple visual help like the one suggested would have helped preventing policy violations.

comment:7 Changed 9 years ago by angel_il

  • Milestone changed from 4.7.0-pre4 to 4.7

comment:8 Changed 9 years ago by angel_il

branch: 1514_over_wrap_line
changeset: aafd9353ab45a00c32ff90a52c6235a6caec5631

comment:9 Changed 9 years ago by angel_il

  • severity changed from no branch to on review
  • Milestone changed from 4.7 to 4.7.1

changeset: ef4b02eeda68d94c18d96087e1f23526ca8719c8

please review

comment:10 Changed 9 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:11 Changed 9 years ago by slavazanko

  • Votes for changeset changed from andrew_b to andrew_b slavazanko
  • severity changed from on review to approved

comment:12 Changed 9 years ago by angel_il

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b slavazanko to committed-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:13 Changed 9 years ago by andrew_b

  • Status changed from testing to closed

comment:14 Changed 9 years ago by angel_il

  • Status changed from closed to reopened
  • Votes for changeset committed-master deleted
  • Resolution fixed deleted

comment:15 Changed 9 years ago by angel_il

reason: http://mail.gnome.org/archives/mc-devel/2010-January/msg00035.html

One with curtsor position at 71'th character, another at 72'th position.

The background suddenly becomes black.

--
Janek Kozicki

screenshot: http://mail.gnome.org/archives/mc-devel/2010-January/pngytUgAUOM8q.png

comment:16 Changed 9 years ago by angel_il

  • severity changed from merged to on review

branch: 1514_right_margin_fix
changeset: b8ee28c2fd806c2ae6a9d1e813b29bb74f285238

comment:17 Changed 9 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Version changed from 4.7.0-pre1 to master
  • Type changed from enhancement to defect

comment:18 Changed 9 years ago by metux

rebased and running through testfarm.

i'll vote if it passes all build-tests.

comment:19 Changed 9 years ago by slavazanko

  • Status changed from reopened to assigned
  • Votes for changeset changed from andrew_b to andrew_b slavazanko
  • severity changed from on review to approved

comment:20 Changed 9 years ago by metux

  • Votes for changeset changed from andrew_b slavazanko to committed-master
  • severity changed from approved to merged

comment:21 Changed 9 years ago by metux

  • Status changed from assigned to testing
  • Resolution set to fixed

comment:23 Changed 9 years ago by angel_il

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.