Ticket #1514 (closed defect: fixed)

Opened 15 years ago

Last modified 14 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 15 years ago.
qt designer with 80 columns delimiter
80.gif (96.6 KB) - added by sergey-feo 15 years ago.
How can it looks in mcedit

Change History

Changed 15 years ago by sergey-feo

qt designer with 80 columns delimiter

Changed 15 years ago by sergey-feo

How can it looks in mcedit

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

Replying to sergey-feo:

How can it looks in mcedit

are you really need this?

comment:2 Changed 15 years ago by angel_il

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

comment:3 Changed 15 years ago by angel_il

  • Milestone changed from future releases to 4.7

comment:4 Changed 15 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 15 years ago by angel_il

  • Milestone changed from 4.7 to 4.7.0-pre4

comment:6 Changed 14 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 14 years ago by angel_il

  • Milestone changed from 4.7.0-pre4 to 4.7

comment:8 Changed 14 years ago by angel_il

branch: 1514_over_wrap_line
changeset: aafd9353ab45a00c32ff90a52c6235a6caec5631

comment:9 Changed 14 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 14 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:11 Changed 14 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 14 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 14 years ago by andrew_b

  • Status changed from testing to closed

comment:14 Changed 14 years ago by angel_il

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

comment:15 Changed 14 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 14 years ago by angel_il

  • severity changed from merged to on review

branch: 1514_right_margin_fix
changeset: b8ee28c2fd806c2ae6a9d1e813b29bb74f285238

comment:17 Changed 14 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 14 years ago by metux

rebased and running through testfarm.

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

comment:19 Changed 14 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 14 years ago by metux

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

comment:21 Changed 14 years ago by metux

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

comment:23 Changed 14 years ago by angel_il

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